Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix regression with columns appender #6237

Merged
merged 1 commit into from
Apr 18, 2018
Merged

Conversation

jasmussen
Copy link
Contributor

This fixes an issue where the default appender in columns would have a border, and the wrong font. This was introduced as part of the input style consistency PR. Increasing selector specificity fixes this.

Before:

screen shot 2018-04-18 at 10 56 26

After:

screen shot 2018-04-18 at 10 57 09

👋 don't mind the yellow background color, I'm experimenting with editor styles :D

This fixes an issue where the default appender in columns would have a border, and the wrong font. This was introduced as part of the input style consistency PR. Increasing selector specificity fixes this.
@jasmussen jasmussen added the [Type] Bug An existing feature does not function as intended label Apr 18, 2018
@jasmussen jasmussen self-assigned this Apr 18, 2018
Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@youknowriad youknowriad merged commit 79c00f9 into master Apr 18, 2018
@youknowriad youknowriad deleted the fix/column-appender branch April 18, 2018 09:07
@phpbits
Copy link
Contributor

phpbits commented Apr 20, 2018

@jasmussen I can't find the changes where the blocks.BlockEdit were disabled on blocks inside the columns. I thought it's this one, would you mind pointing me to the right direction?

I've created this plugin : http://wordpress.org/plugins/block-options/ and the options I've added are not showing on the inner columns anymore. Thanks!

@jasmussen
Copy link
Contributor Author

Cool plugin!

I don't know that I'm the right one to ask (at least I don't know how to answer). If you can, can you ask your question in the WordPress core editor chat? https://make.wordpress.org/chat/ — channel: #core-editor — you can ping me, username @joen in slack, and I'll boost your signal.

@phpbits
Copy link
Contributor

phpbits commented Apr 20, 2018

Thanks @jasmussen :) Really appreciated!

nuzzio pushed a commit to nuzzio/gutenberg that referenced this pull request Apr 25, 2018
This fixes an issue where the default appender in columns would have a border, and the wrong font. This was introduced as part of the input style consistency PR. Increasing selector specificity fixes this.
@mtias mtias modified the milestone: 2.8 May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants