Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#49174 closed defect (bug) (fixed)

Incorrect usage of <label> in wp-signup.php

Reported by: audrasjb Owned by: audrasjb
Priority: normal Milestone: 5.4
Component: Login and Registration Version:
Severity: normal Keywords: good-first-bug has-patch commit
Cc: Focuses: accessibility, multisite

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)

49174.diff​ (508 bytes ) - added by vaishalipanchal 7 years ago.
Patch.

Download all attachments as: .zip

Change History (4)

@vaishalipanchal
7 years ago

Patch.

#1 @vaishalipanchal
7 years ago

  • Keywords has-patch added; needs-patch removed

#2 @audrasjb
7 years ago

  • Keywords commit added; needs-testing removed
  • Milestone Awaiting Review → 5.4
  • Owner set to audrasjb
  • Status new → 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 :-)

#3 @SergeyBiryukov
7 years ago

  • Resolution → fixed
  • Status accepted → closed

In 47074:

Login and Registration: Remove duplicate <label> tag from show_blog_form() in wp-signup.php.

Props vaishalipanchal, audrasjb.
Fixes #49174.

Note: See TracTickets for help on using tickets.