Skip to content

core_widgets example slider fix #19810

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

Merged

Conversation

ickshonpe
Copy link
Contributor

@ickshonpe ickshonpe commented Jun 25, 2025

Objective

When dragging the slider thumb the thumb is only highlighted while the pointer is hovering the widget. If the pointer moves off the widget during a drag the thumb reverts to its normal unhovered colour.

Solution

Query for CoreSliderDragState in the slider update systems and set the lighter color if the thumb is dragged or hovered.

…lour of the thumb so that it keeps the active colour during drags even when the pointer is not hovering the slider.
@ickshonpe ickshonpe changed the title core_widgets example slider fix core_widgets example slider fix Jun 25, 2025
@ickshonpe ickshonpe requested a review from viridia June 25, 2025 10:40
@ickshonpe ickshonpe added C-Examples An addition or correction to our examples A-UI Graphical user interfaces, styles, layouts, and widgets D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jun 25, 2025
@jordanhalase
Copy link

Does not seem to conflict with #19792 🙂

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jun 29, 2025
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jun 29, 2025
Merged via the queue into bevyengine:main with commit 8d70649 Jun 29, 2025
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-UI Graphical user interfaces, styles, layouts, and widgets C-Examples An addition or correction to our examples D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants