Opened 6 years ago
Closed 6 years ago
#49174 closed defect (bug) (fixed)
Incorrect usage of <label> in wp-signup.php
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 5.4 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Login and Registration | Keywords: | good-first-bug has-patch commit |
| Focuses: | accessibility, multisite | Cc: |
Description
There is a duplicate/wrong use of <label> in wp-signup.php:
https://core.trac.wordpress.org/browser/branches/5.3/src/wp-signup.php#L201
The text "Privacy:" is not a label but rather a section title in my opinion. Ideally, it could be transformed into a proper heading. By the way, a very quick fix would be to simply remove the <label> HTML tag that wraps this text.
Attachments (1)
Change History (4)
#2
@
6 years ago
- Keywords commit added; needs-testing removed
- Milestone changed from Awaiting Review to 5.4
- Owner set to audrasjb
- Status changed from new to accepted
Thanks for the patch @vaishalipanchal, looks good to me.
Adding commit keyword and milestoning to 5.4.
Please note: we use good-first-bug keyword to identify easy bugfixes so new contributors can learn how to make a patch and contribute to WP Core. It's a very helpful way to onboard new contributors, so please remember to give them priority on those :-)
Patch.