-
Notifications
You must be signed in to change notification settings - Fork 22
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
[d2l-meter] property for hiding text #4587
base: main
Are you sure you want to change the base?
Conversation
Thanks for the PR! 🎉 We've deployed an automatic preview for this PR - you can see your changes here:
Note The build needs to finish before your changes are deployed. |
@@ -57,26 +74,33 @@ class MeterCircle extends MeterMixin(RtlMixin(LitElement)) { | |||
|
|||
const primary = this._primary(this.value, this.max) || ''; | |||
const secondary = this._secondary(this.value, this.max, this.text); | |||
const secondaryTextElement = (this.text && !this.textHidden) ? html`<div class="d2l-body-small d2l-meter-circle-text-secondary">${secondary}</div>` : nothing; |
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.
there are 3 uses where we need to add text-hidden before we merge this https://d2l.slack.com/archives/C0PHG3QB0/p1712758960144389?thread_ts=1712758381.367159&cid=C0PHG3QB0
Context
text
property like the radial meter, however it only assigned it to the aria text. We'd like the circle meter to be able to behave similarly to the radial meterImplementation Details
text-hidden
prop to meter-mixin