-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add v-align-baseline #324
Add v-align-baseline #324
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you! Just one tiny nitpick comment but otherwise good to go!
.direction-rtl { direction: rtl !important; } | ||
|
||
/* Set the direction to ltr */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tiny nitpick, can you remove the whitespace line above to be consistent with other families of utilities?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem! Happy to be more consistent. Change has been made 👍
@jonrohan @shawnbot I'm merging this into dev so that it goes into the next release. It's possible that since this utility was documented but didn't exist that people have applied it without it working—so I searched |
Add the
.v-align-baseline
class, which is avertical-align: baseline
utility class.