Skip to content

feathers_gallery infinite update loop fix#23827

Merged
alice-i-cecile merged 4 commits intobevyengine:mainfrom
ickshonpe:feathers-gallery-color-change-detection-fix
Apr 17, 2026
Merged

feathers_gallery infinite update loop fix#23827
alice-i-cecile merged 4 commits intobevyengine:mainfrom
ickshonpe:feathers-gallery-color-change-detection-fix

Conversation

@ickshonpe
Copy link
Copy Markdown
Contributor

Objective

In the feathers_gallery example, on a TextEditChange handle_hex_color_change updates DemoWidgetStates which causes update_colors to update the hex input triggering another TextEditChange.

Solution

Only update DemoWidgetStates in handle_hex_color_change if the new color is different.

On a `TextEditChange` `handle_hex_color_change` updates `DemoWidgetStates` which causes `update_colors` to update the hex input triggering another `TextEditChange`.

This commit fixes the problem by only updating `DemoWidgetStates` if the color value is different.
@ickshonpe ickshonpe changed the title feathers_gallery example infinite update loop fix feathers_gallery infinite update loop fix Apr 16, 2026
@ickshonpe ickshonpe added C-Examples An addition or correction to our examples C-Bug An unexpected or incorrect behavior A-UI Graphical user interfaces, styles, layouts, and widgets labels Apr 16, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in UI Apr 16, 2026
@ickshonpe ickshonpe added D-Trivial Nice and easy! A great choice to get started with Bevy S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Apr 16, 2026
@Zeophlite Zeophlite added this to the 0.19 milestone Apr 16, 2026
@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 Apr 16, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 17, 2026
Merged via the queue into bevyengine:main with commit 4c7e99c Apr 17, 2026
38 checks passed
@github-project-automation github-project-automation Bot moved this from Needs SME Triage to Done in UI Apr 17, 2026
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-Bug An unexpected or incorrect behavior C-Examples An addition or correction to our examples D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants