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

Multiple themes: Double-check quote styles with GB 5.2 #594

Closed
laurelfulford opened this issue Feb 12, 2019 · 2 comments
Closed

Multiple themes: Double-check quote styles with GB 5.2 #594

laurelfulford opened this issue Feb 12, 2019 · 2 comments
Assignees
Labels
[Pri] High [Type] Bug Something isn't working
Milestone

Comments

@laurelfulford
Copy link
Contributor

laurelfulford commented Feb 12, 2019

The Quote block is going to get a bit of a style refresh in a future release of Gutenberg (5.2, milestoned for March 4th). If the block's text is either left or right aligned, a border is added to that side of the block:

WordPress/gutenberg#13248

This can potentially cause visual issues if a theme already has a side border defined for this block. So we need to double-check the themes that have Gutenberg styles, to see if there are any clashes, and proactively fix them.

Themes to check:

It's important to note that this did cause an issue with Twenty Nineteen so any of its child themes will likely have the same problem; it will need to be fixed in each of those themes individually, since the styles aren't inherited from the parent theme.

@laurelfulford laurelfulford added [Type] Bug Something isn't working [Pri] High labels Feb 12, 2019
@laurelfulford laurelfulford added this to the All themes milestone Feb 12, 2019
@laurelfulford laurelfulford self-assigned this Feb 12, 2019
@laurelfulford laurelfulford changed the title Multiple themes: Double-check pullquote styles witjh GB 5.2 Multiple themes: Double-check pullquote styles with GB 5.2 Feb 12, 2019
@laurelfulford laurelfulford changed the title Multiple themes: Double-check pullquote styles with GB 5.2 Multiple themes: Double-check quote styles with GB 5.2 Feb 12, 2019
laurelfulford added a commit that referenced this issue Feb 14, 2019
…the new styles planned for Gutenberg 5.2. See #594.
@laurelfulford
Copy link
Contributor Author

Note that the color-patterns.php changes being added to Twenty Nineteen need to be synced up with the WP.com version after they're committed, to be made available in the style variations themes (Calm, Elegant, Friendly, Modern, Professional & Sophisticated). But the CSS/Sass updates can be added to each theme now.

laurelfulford added a commit that referenced this issue Feb 14, 2019
…th the new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 14, 2019
…ith the new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 14, 2019
…h the new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 14, 2019
…er with the new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 14, 2019
…ter with the new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 14, 2019
laurelfulford added a commit that referenced this issue Feb 14, 2019
…he new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 14, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
…ter with the new styles planned for Gutenberg 5.2. See #594.
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
laurelfulford added a commit that referenced this issue Feb 15, 2019
@laurelfulford
Copy link
Contributor Author

These should all be taken care of now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] High [Type] Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant