fix(button): blur button to remove pressed state after modal opens (#… #8520
+5,233
−4,566
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…8339)
This PR fixes issue #8339 where a button remains in a data-pressed="true" state after opening a modal using DialogTrigger.
The fix ensures that the button blurs on click, removing the pressed state immediately after interaction.
Behavior Before
Clicking a button that opens a modal caused it to remain visually "pressed"
data-pressed="true" would persist even while the modal was open
This created a distracting UI effect, especially with outlined/animated buttons
Behavior After
The button now calls .blur() on click
The data-pressed attribute is removed as expected
The visual feedback resets properly after modal opens
Fix Location
The fix was added directly to the JSX inside packages/@react-spectrum/button/src/Button.tsx by adding an onClick handler:
onClick={(e) => {
e.currentTarget.blur();
props.onClick?.(e);
}}
Existing button behavior and modal triggering remain unaffected
No breaking changes introduced