mirror of
https://github.com/fosrl/pangolin.git
synced 2026-05-08 05:39:49 -05:00
[GH-ISSUE #1686] Rule priority doesn't update properly #6772
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/pangolin#6772
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @matt01y on GitHub (Oct 17, 2025).
Original GitHub issue: https://github.com/fosrl/pangolin/issues/1686
Originally assigned to: @Pallavikumarimdb on GitHub.
Describe the Bug
If only the up/down arrow are used to change the priority integer. The priorities don't update when saving.
Environment
To Reproduce
When updating rule priorities by only pressing the up/down arrows the priorities don't update when pressing
save all settingsExcept when you first put your cursor (mouse click on cell) again on the number.
Expected Behavior
Values update when just using the up/down arrows.
@oschwartz10612 commented on GitHub (Oct 17, 2025):
@Pallavikumarimdb could you look at this one as well? I bet this is react related because the input has not been "touched".
@Pallavikumarimdb commented on GitHub (Oct 17, 2025):
@oschwartz10612 sure👍
@Pallavikumarimdb commented on GitHub (Oct 18, 2025):
This isn’t a bug - it’s expected browser behavior.
The up/down arrows on a number input don’t trigger a change event unless the input is focused.
You need to first click inside the field (to focus it) before using the arrows for the updated value to be registered when saving.
I’ve added an improvement (onClick={(e) => e.currentTarget.focus()}) to ensure the input auto-focuses when clicked, so the behavior feels more intuitive.
@oschwartz10612 commented on GitHub (Oct 19, 2025):
Thanks @Pallavikumarimdb appreciate the analysis!