Skip to content

[slider] Align value label text center#23075

Merged
oliviertassinari merged 3 commits into
mui:nextfrom
LorenzHenk:slider-align-label
Oct 15, 2020
Merged

[slider] Align value label text center#23075
oliviertassinari merged 3 commits into
mui:nextfrom
LorenzHenk:slider-align-label

Conversation

@LorenzHenk

Copy link
Copy Markdown

The Slider value label was left aligned (default).
This PR changes the alignment to be center.

Before:
Bildschirmfoto am 2020-10-15 um 20 25 38

After:
Bildschirmfoto am 2020-10-15 um 20 25 29

@oliviertassinari oliviertassinari added type: bug It doesn't behave as expected. scope: slider Changes related to the slider. labels Oct 15, 2020

@oliviertassinari oliviertassinari left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@LorenzHenk

Copy link
Copy Markdown
Author

We need to fix next.material-ui.com/components/slider-styled too

Sure, I can do that 👍

@mui-pr-bot

mui-pr-bot commented Oct 15, 2020

Copy link
Copy Markdown

Details of bundle changes

Generated by 🚫 dangerJS against a47dbb8

Comment thread packages/material-ui-lab/src/SliderStyled/ValueLabelStyled.js Outdated
@oliviertassinari
oliviertassinari merged commit a92256e into mui:next Oct 15, 2020
@oliviertassinari

Copy link
Copy Markdown
Member

@LorenzHenk Thanks

@LorenzHenk
LorenzHenk deleted the slider-align-label branch October 16, 2020 07:03
@oliviertassinari oliviertassinari changed the title [Slider] Align value label text center [slider] Align value label text center May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: slider Changes related to the slider. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants