WordPress.org

Make WordPress Core

Opened 9 months ago

Closed 9 months ago

Last modified 5 months ago

#47355 closed defect (bug) (fixed)

Opening/closing tag mismatch in media templates

Reported by: TravisSeitler Owned by: SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version: 5.0
Component: Media Keywords:
Focuses: Cc:
PR Number:

Description

When looking into another issue I realized both the #tmpl-audio-details and #tmpl-video-details templates in [src/wp-includes/media-template.php] have mismatched tags for input field labels. (In both cases, the opening tag is a <label> but the closing tag is a </span>.)

#tmpl-audio-details: https://core.trac.wordpress.org/browser/trunk/src/wp-includes/media-template.php#L1138
#tmpl-video-details: https://core.trac.wordpress.org/browser/trunk/src/wp-includes/media-template.php#L1273

Attachments (1)

47355.diff (1.5 KB) - added by TravisSeitler 9 months ago.

Download all attachments as: .zip

Change History (6)

@TravisSeitler
9 months ago

#1 @SergeyBiryukov
9 months ago

  • Component changed from General to Login and Registration
  • Milestone changed from Awaiting Review to 5.3
  • Version set to 5.0

Hi @TravisSeitler, welcome to WordPress Trac! Thanks for the ticket.

Introduced in [42444].

#2 @SergeyBiryukov
9 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 45381:

Media: Correct closing </label> tags in #tmpl-audio-details and #tmpl-video-details after [42444].

Props TravisSeitler.
Fixes #47355.

#3 @SergeyBiryukov
9 months ago

  • Component changed from Login and Registration to Media

#4 @afercia
9 months ago

Thanks for the ticket and the fix :) @SergeyBiryukov quick question: should tests/qunit/index.html be updated or it's considered legacy and not really used any longer?

#5 @desrosj
5 months ago

#48159 was marked as a duplicate.

Note: See TracTickets for help on using tickets.