Skip to content

Don't set display: none on elements that use hidden="until-found"#14631

Merged
RobinMalfait merged 3 commits into
nextfrom
fix/improve-hidden-until-found
Oct 9, 2024
Merged

Don't set display: none on elements that use hidden="until-found"#14631
RobinMalfait merged 3 commits into
nextfrom
fix/improve-hidden-until-found

Conversation

@RobinMalfait

Copy link
Copy Markdown
Member

Fixes an issue reported by the React Aria Components team here:

adobe/react-spectrum#7160

Basically hidden="until-found" behaves very differently than hidden and doesn't actually use display: none, so we don't want to apply the behavior we apply for the regular hidden attribute.

Fixes an issue reported by the React Aria Components team here:

adobe/react-spectrum#7160

Basically `hidden="until-found"` behaves very differently than `hidden` and doesn't actually use `display: none`, so we don't want to apply the behavior we apply for the regular `hidden` attribute.
@RobinMalfait RobinMalfait added the v4 label Oct 9, 2024
@RobinMalfait RobinMalfait merged commit b1733ac into next Oct 9, 2024
@RobinMalfait RobinMalfait deleted the fix/improve-hidden-until-found branch October 9, 2024 11:56
RobinMalfait added a commit that referenced this pull request Oct 9, 2024
…#14631)

Fixes an issue reported by the React Aria Components team here:

adobe/react-spectrum#7160

Basically `hidden="until-found"` behaves very differently than `hidden`
and doesn't actually use `display: none`, so we don't want to apply the
behavior we apply for the regular `hidden` attribute.
tongrow pushed a commit to tongrow/tailwind-normalize that referenced this pull request Jun 2, 2025
…tailwindlabs#14631)

Fixes an issue reported by the React Aria Components team here:

adobe/react-spectrum#7160

Basically `hidden="until-found"` behaves very differently than `hidden`
and doesn't actually use `display: none`, so we don't want to apply the
behavior we apply for the regular `hidden` attribute.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants