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(material-experimental/mdc-slider): append class to indicate short… #23566
fix(material-experimental/mdc-slider): append class to indicate short… #23566
Conversation
@jelbourn cl/388828896 is going to take a while to get in. This is a temporary fix while that CL is pending. I've already submitted cl/395925245 in preparation for this fix |
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.
LGTM
Can add merge ready when ready
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. ℹ️ Googlers: Go here for more info. |
… value indicator text * This class is only used internally. It is a temporary fix which will be removed once the appending of this class is handled by the mdc foundation (cl/388828896).
70a5557
to
23e6874
Compare
@@ -150,6 +154,11 @@ export class MatSliderVisualThumb implements AfterViewInit, OnDestroy { | |||
this._elementRef.nativeElement.removeEventListener('mouseleave', this._onMouseLeave); | |||
} | |||
|
|||
/** Used to append a class to indicate when the value indicator text is short. */ | |||
_isShortValue(): boolean { | |||
return this.valueIndicatorText?.length <= 2; |
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.
nit: it might be a good idea to initialize the valueIndicatorText
to an empty string so that we don't have to null check it everywhere. That would also make it easier to inline the entire expression in the host
binding.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
… value indicator text
once the appending of this class is handled by the mdc foundation (cl/388828896).