Skip to content

fix(ui): disable slider arrows at limits - #968

Merged
Wyvest merged 2 commits into
Polyfrost:v1from
saadndm:fix-arrows
Aug 16, 2026
Merged

fix(ui): disable slider arrows at limits#968
Wyvest merged 2 commits into
Polyfrost:v1from
saadndm:fix-arrows

Conversation

@saadndm

@saadndm saadndm commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Description

  • Overloaded Modifier.onClick to allow passing enabled (Maybe theres a better way to do this?)
  • Disabled slider spinner arrow at limits, to stop misleading hover and click feedback

Before:

before.mp4

After:

635829668-cb626450-f923-4359-902c-d460c1d40da6.mp4

Related Issues

Fixes #735

Checklist

  • I made a clear description of what was changed
  • I stated why these changes were necessary
  • I updated documentation or said what needs to be updated
  • I made sure these changes are backwards compatible
  • This pull request is for one feature/bug fix

@Wyvest
Wyvest merged commit 0e73a45 into Polyfrost:v1 Aug 16, 2026
1 check passed
@saadndm
saadndm deleted the fix-arrows branch August 16, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

no feedback for when slider has hit max or min value when using arrow buttons

2 participants