-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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(text-field): Adjust the baseline of text field's helper text to match spec #3069
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3069 +/- ##
==========================================
+ Coverage 98.24% 98.35% +0.11%
==========================================
Files 120 120
Lines 5129 5118 -11
Branches 638 638
==========================================
- Hits 5039 5034 -5
+ Misses 90 84 -6
Continue to review full report at Codecov.
|
🤖 Beep boop! Screenshot test reportCommit b132601 vs. No diffs! 💯🎉 |
demos/text-field.html
Outdated
aria-hidden="true" style="display:none;"> | ||
Helper Text (possibly validation message) | ||
aria-hidden="true" style="display:none;" | ||
>Helper Text (possibly validation message) |
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.
Why is the helper text sensitive to leading whitespace now? AFAICT it was not previously?
If this is a limitation of the new baseline styles/mixin, I'd like to see us work it out if at all possible...
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.
One potential cleanup item, otherwise LGTM (don't forget BREAKING CHANGE: ...
in the actual squash commit description)
<p id="username-helper-text" class="mdc-text-field-helper-text" aria-hidden="true"> | ||
This will be displayed on your public profile | ||
</p> | ||
<p id="username-helper-text" class="mdc-text-field-helper-text" aria-hidden="true">This will be displayed on your public profile</p> |
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.
I guess this formatting change can be reverted?
…xt_baseline_issue2783
Fixes #2783
BREAKING CHANGE
Changes included: