-
Notifications
You must be signed in to change notification settings - Fork 112
8366278: Form control element <select> has no associated label #375
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Pasam Soujanya <psoujany@in.ibm.com>
|
👋 Welcome back psoujany! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
|
Fix Request : https://bugs.openjdk.org/browse/JDK-8366278 request clean backport, Form control element |
|
@hns Could you please check on this backport. Thank you. |
|
The change looks good, but the issue is marked as fixed in 25. If this is by mistake, it should be fixed before doing the backport. Also, there are some rules regarding backporting that must be followed, including getting approval for a backport by the maintainer. |
|
I mistakenly set version to 25. Thank you for clarifying @hns |
|
The "Issue" section above explains how to make this a backport pull request by setting the PR title to "Backport [hash]" where [hash] is the hash of the original commit in the main repository. |
|
This backport pull request has now been updated with issue from the original commit. |
|
|
|
/approval request for backport of |
|
Hi @psoujany |
|
Hi @GoeLin , the fix is related to accessibility. We tested the fix with JDK25 and didn't find any regressions, so this could be of low risk. Could you please let me know if anything needs to be validated for this backport. Thank you. |
|
Hi @psoujany, |
This PR is a backport of https://bugs.openjdk.org/browse/JDK-8366278
Progress
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk25u.git pull/375/head:pull/375$ git checkout pull/375Update a local copy of the PR:
$ git checkout pull/375$ git pull https://git.openjdk.org/jdk25u.git pull/375/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 375View PR using the GUI difftool:
$ git pr show -t 375Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk25u/pull/375.diff
Using Webrev
Link to Webrev Comment