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(block): Ensure description has correct line-height #6723

Merged

Conversation

macandcheese
Copy link
Contributor

@macandcheese macandcheese commented Apr 4, 2023

Related Issue: #6443

Summary

The previous Tailwind text declaration wasn't setting a line-height, meaning the description could unintentionally inherit line-height from a parent container. This wasn't affecting the collapsible block as the rendered button has additional style set that prevents the issue.

Before After
Screen Shot 2023-04-03 at 5 53 38 PM Screen Shot 2023-04-03 at 5 53 47 PM

@macandcheese macandcheese requested a review from a team as a code owner April 4, 2023 01:00
@github-actions github-actions bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label Apr 4, 2023
@macandcheese macandcheese added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Apr 4, 2023
@jcfranco jcfranco added pr ready for visual snapshots Adding this label will run visual snapshot testing. and removed pr ready for visual snapshots Adding this label will run visual snapshot testing. labels Apr 5, 2023
Copy link
Member

@jcfranco jcfranco left a comment

Choose a reason for hiding this comment

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

💥

@macandcheese
Copy link
Contributor Author

@SkyeSeitz @ashetland - please check these Chromatic changes - https://www.chromatic.com/pullrequest?appId=6266d45509d7eb004aa254fb&number=6723&activeElementId=comparison-test-key-636c55c5698b291127b1a60d-1200&view=changes - this update adjusts by a few pixels. Based on the original issue, this is an expected change with this update, but want to confirm.

@macandcheese
Copy link
Contributor Author

Merging as Chromatic changes have been approved - note - there are slight px adjustment as a result.

@macandcheese macandcheese merged commit a7deec2 into master Apr 6, 2023
7 checks passed
@macandcheese macandcheese deleted the macandcheese/6443-block-description-line-height branch April 6, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports for broken functionality. Issues should include a reproduction of the bug. pr ready for visual snapshots Adding this label will run visual snapshot testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants