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

Improve button block styles and wrap. #12205

Merged
merged 1 commit into from
Nov 23, 2018
Merged

Improve button block styles and wrap. #12205

merged 1 commit into from
Nov 23, 2018

Conversation

jasmussen
Copy link
Contributor

This PR fixes #7843, and also simplifies the styles a bit. It also fixes an overflow issue with long text in variations. Finally, it makes a single line button have a proper pillshape.

Before:

screenshot 2018-11-22 at 09 52 53

screenshot 2018-11-22 at 09 53 36

After:

screenshot 2018-11-22 at 10 04 03

screenshot 2018-11-22 at 09 59 59

This PR fixes #7843, and also simplifies the styles a bit. It also fixes an overflow issue with long text in variations. Finally, it makes a single line button have a proper pillshape.
@jasmussen jasmussen added the [Type] Enhancement A suggestion for improvement. label Nov 22, 2018
@jasmussen jasmussen added this to the WordPress 5.0.x Follow Ups milestone Nov 22, 2018
@jasmussen jasmussen self-assigned this Nov 22, 2018
@jasmussen jasmussen requested a review from a team November 22, 2018 09:07
Copy link
Member

@jorgefilipecosta jorgefilipecosta left a comment

Choose a reason for hiding this comment

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

I did some tests and I did not find any problem with the new styles 👍 The code changes make sense.

@mtias mtias modified the milestones: WordPress 5.0.x Follow Ups, 4.7 Nov 22, 2018
@jasmussen
Copy link
Contributor Author

Same as with #12212 (comment), I'm leaving unmerged as I'm unsure when the correct time to merge is. Feel free to merge if the time is right.

@youknowriad youknowriad merged commit 1f57485 into master Nov 23, 2018
@youknowriad youknowriad deleted the fix/button-wrap branch November 23, 2018 09:00
@ZebulanStanphill
Copy link
Member

Should this be moved out of the "4.7" and "WordPress 5.0.x Follow Ups" milestones and into the "4.6" milestone?

@mtias mtias modified the milestones: 4.7, 4.6 Nov 28, 2018
@mtias
Copy link
Member

mtias commented Nov 28, 2018

Yes.

youknowriad pushed a commit that referenced this pull request Nov 29, 2018
This PR fixes #7843, and also simplifies the styles a bit. It also fixes an overflow issue with long text in variations. Finally, it makes a single line button have a proper pillshape.
@mtias mtias added the [Block] Buttons Affects the Buttons Block label Nov 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Buttons Affects the Buttons Block [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Button block: text should word-wrap if needed
5 participants