Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: Backport the hidden="until-found" attr tests from 3.x-stable #5619

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

mgol
Copy link
Member

@mgol mgol commented Jan 27, 2025

Summary

The hidden attribute used to be a boolean one but it gained a new until-found eventually. This led us to change the way we handle boolean attributes in jQuery 4.0 in gh-5452 to avoid these issues in the future.

We haven't added an explicit test for the "until-found" value of the hidden attribute which triggered this decision so far, though. Backport the test from gh-5607 which landed on 3.x-stable so that we do test it.

Ref gh-5452
Ref gh-5607

(cherry picked from commit 85290c5)

Checklist

Sorry, something went wrong.

The `hidden` attribute used to be a boolean one but it gained a new
`until-found` eventually. This led us to change the way we handle boolean
attributes in jQuery 4.0 in jquerygh-5452 to avoid these issues in the future.

We haven't added an explicit test for the `"until-found"` value of the
`hidden` attribute which triggered this decision so far, though.
Backport the test from jquerygh-5607 which landed on `3.x-stable` so that we
do test it.

Ref jquerygh-5452
Ref jquerygh-5607

(cherry picked from commit 85290c5)
@mgol mgol self-assigned this Jan 27, 2025
@mgol mgol marked this pull request as ready for review January 27, 2025 17:39
@mgol mgol added this to the 4.0.0 milestone Feb 24, 2025
@mgol mgol merged commit 3a31866 into jquery:main Feb 24, 2025
17 checks passed
@mgol mgol deleted the until-found-tests branch February 24, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants