-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[Slider] Allow individual mark customization #17057
Conversation
Details of bundle changes.Comparing: 078fd1e...b1c592c Details of page changes
|
data-index
to MarkLabeldata-index
to mark labels
Considering that we already have global class names, and never used DOM attributes yet for encouraging customization, after more thoughts, I propose a different approach. |
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.
Thanks for working on this issue. Could you please add a test so that we don't remove this feature in the future accidentally?
Could you also provide an example that would make use of this feature and explain why :nth-of-type
would not work for that case?
Interesting, I haven't thought about this option. With |
This is why a concrete use case would be nice: so that we could explore other implementations. I think you would use something It would even work for what was mentioned in the issue: Targetting the last label with |
Hi @oliviertassinari @eps1lon ! EDIT:
I tried to use that pseudo, but I didn't get that working through MUI Theme override. I will appreciate if we have data-index or classname+index and I think could be very useful for other users. How could I help you to get this integrated? Submitting a testcase? Trying something else? Thanks! |
I am interested in this issue, but I'm having trouble figuring out what was the solution? |
@Louis345 What's your use case? :) |
@oliviertassinari Thank you so much for your prompt response. I would like to have the ability to move an individual label. On mobile with limited width, some of the labels goes off the screen. Here is an image: |
9b600c3
to
308c2c1
Compare
data-index
to mark labels19858e3
to
b1c592c
Compare
This is great. |
How do I get it into my code? Will there be another release? |
Closes #17023