fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger#2279
Conversation
Changed Packages
|
|
/hold |
|
Test Spinner ActiveText: Minimal workflow to validate ActiveText shows a spinner immediately when retrigger inputs change. test-spinner-active-text.sw.yaml schemas/test-spinner-active-text__main-schema.json |
|
Test Spinner ActiveDropdown Static dropdown to trigger ActiveText fetch on selection change without dropdown fetching. test-spinner-active-dropdown.sw.yaml schemas/test-spinner-active-dropdown__main-schema.json |
Co-authored-by: Cursor <cursoragent@cursor.com>
Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com>
Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com>
Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com>
53c8e9d to
2d96cc0
Compare
|
|
------BEFORE------- Screen.Recording.2026-02-25.at.4.09.52.PM.mov--------AFTER------ Screen.Recording.2026-02-25.at.3.47.11.PM.mov |
|
/unhold |
|
Thanks, it was tricky issue. It's good that this behavior is left to be optional, there are cases where the old flow makes sense. |
|
/cherrypick orchestrator-1.8 |
|
@lokanandaprabhu: #2279 failed to apply on top of branch "orchestrator-1.8": DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |





Hey, I just made a Pull Request!
Fixes:
https://issues.redhat.com/browse/RHDHBUGS-2594
PR description:
Problem
ActiveText widgets with
fetch:retrigger(and nofetch:url) briefly show staleui:textduring the debounce window. The spinner only appears once evaluation starts, which is visible in production latency.Solution
fetch:clearOnRetriggeris enabled to avoid stale state during refetch.Changes
fetch:clearOnRetriggersupport forActiveTextInput,ActiveDropdown, andActiveMultiSelect.✔️ Checklist