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 formatting button hover/focus style #6493

Merged
merged 1 commit into from
Apr 30, 2018

Conversation

jasmussen
Copy link
Contributor

A recent fix to aria-disabled buttons (up mover on first block) caused a regression in formatting buttons. This PR fixes that.

Screenshot, after:

screen shot 2018-04-30 at 08 58 30

Don't mind the background color, that's from the theme and unrelated to this PR.

A recent fix to aria-disabled buttons (up mover on first block) caused a regression in formatting buttons. This PR fixes that.
@jasmussen jasmussen added the [Type] Bug An existing feature does not function as intended label Apr 30, 2018
@jasmussen jasmussen self-assigned this Apr 30, 2018
@jasmussen jasmussen requested a review from a team April 30, 2018 07:00
@jasmussen
Copy link
Contributor Author

Wohoo, thank you!

@jasmussen jasmussen merged commit da36107 into master Apr 30, 2018
@jasmussen jasmussen deleted the fix/formatting-hover-regression branch April 30, 2018 08:54
@danielbachhuber danielbachhuber added this to the 2.8 milestone Apr 30, 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.

3 participants