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(fab): Set fab label line-height to normal to center text #5258

Merged
merged 6 commits into from
Nov 18, 2019

Conversation

EstebanG23
Copy link
Contributor

No description provided.

"screenshots": {
"desktop_windows_chrome@77": "https://storage.googleapis.com/mdc-web-screenshot-tests/esgonzalez/2019/11/14/22_24_39_633/spec/mdc-fab/mixins/extended-padding.html.windows_chrome_77.png",
"desktop_windows_chrome@77": "https://storage.googleapis.com/mdc-web-screenshot-tests/esgonzalez/2019/11/15/21_14_53_236/spec/mdc-fab/mixins/extended-padding.html.windows_chrome_77.png",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Alignment in previous screenshot looks more correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Its reverting back to the originals. This for some reason only effects windows 10 however this fix of line-height: 1, doesn't break internal clients.

@EstebanG23 EstebanG23 changed the title Fix(fab): Set line-height to 1 to center text fix(fab): Set line-height to 1 to center text Nov 15, 2019
@EstebanG23 EstebanG23 changed the title fix(fab): Set line-height to 1 to center text fix(fab): Set fab label line-height to 1 to center text Nov 15, 2019
@mdc-web-bot
Copy link
Collaborator

All 731 screenshot tests passed for commit eca9627 vs. master! 💯🎉

@EstebanG23 EstebanG23 merged commit 591a6ad into master Nov 18, 2019
@EstebanG23 EstebanG23 deleted the fix/fab-lineheight branch November 18, 2019 23:41
@EstebanG23 EstebanG23 changed the title fix(fab): Set fab label line-height to 1 to center text fix(fab): Set fab label line-height to normal to center text Nov 21, 2019
This was referenced Feb 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants