Skip to content

feat: add minMaxVisible property to slider and range-slider#11057

Merged
web-padawan merged 1 commit intomainfrom
feat/slider-min-max-limits
Feb 9, 2026
Merged

feat: add minMaxVisible property to slider and range-slider#11057
web-padawan merged 1 commit intomainfrom
feat/slider-min-max-limits

Conversation

@web-padawan
Copy link
Copy Markdown
Member

@web-padawan web-padawan commented Feb 4, 2026

Description

Part of #10973

Add a minMaxVisible boolean property that displays the min and max values below the slider track when enabled.

Type of change

  • Feature

Note

This adds min and max shadow parts wrapped in marks that can be then reused to render tickmarks.

@web-padawan web-padawan requested a review from vursen February 4, 2026 13:40
@web-padawan web-padawan force-pushed the feat/slider-min-max-limits branch from 9754a81 to 8c88e03 Compare February 5, 2026 13:15
* `thumb` | The slider thumb (applies to both thumbs)
* `thumb-start` | The start (lower value) thumb
* `thumb-end` | The end (upper value) thumb
* `limits` | Container for min/max limit labels
Copy link
Copy Markdown
Contributor

@vursen vursen Feb 5, 2026

Choose a reason for hiding this comment

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

The term "limit" seems not ideal when considering future features. With future support for visible steps, this container would likely also be used for holding steps, and min max would just appear as part of that set.

Maybe "marks" would work better?

  1. marks
  2. mark
  3. min-mark
  4. max-mark
  5. step-mark (future)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

IMO, marks is more about "tickmarks" (visual representation of the step). I'll check other implementations for some ideas. Overall I'm not really sure about the API name, let's discuss it tomorrow.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated to use minMaxVisible as discussed.

Add a minMaxVisible boolean property that displays the min and max
values below the slider track when enabled.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@web-padawan web-padawan force-pushed the feat/slider-min-max-limits branch from 8c88e03 to 4884c2a Compare February 9, 2026 09:37
@web-padawan web-padawan changed the title feat: add limitsVisible property to slider and range-slider feat: add minMaxVisible property to slider and range-slider Feb 9, 2026
@web-padawan web-padawan requested a review from vursen February 9, 2026 09:38
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 9, 2026

@web-padawan web-padawan merged commit c4d2fb1 into main Feb 9, 2026
10 checks passed
@web-padawan web-padawan deleted the feat/slider-min-max-limits branch February 9, 2026 09:52
@vaadin-bot
Copy link
Copy Markdown
Collaborator

This ticket/PR has been released with Vaadin 25.1.0-alpha8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants