WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#51473 closed defect (bug) (fixed)

<ul> tag missing in tests/qunit/index.html

Reported by: ravipatel's profile ravipatel Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.6 Priority: normal
Severity: normal Version: 4.6
Component: Build/Test Tools Keywords: has-patch
Focuses: coding-standards Cc:

Description


Attachments (1)

51473-index.html.patch​ (661 bytes) - added by ravipatel 5 years ago.
Ui tag missing

Download all attachments as: .zip

Change History (8)

@ravipatel
5 years ago

Ui tag missing

This ticket was mentioned in ​Slack in #core by ravi. ​View the logs.
5 years ago

#2 @JeffPaul
5 years ago

Looks like a stray </ul> on line 2305 and </li> on line 2304 can be removed perhaps? I haven't looked at the larger file but a quick scan of the changes seems like those may be errant?

#3 @ravipatel
5 years ago

@JeffPaul yes because of 2305 </ul> is inner ul tag which start from 2293.
Please review whole file & compare. I have shared right patch.

See exiting code without patch : https://prnt.sc/uvkb0m

#4 @SergeyBiryukov
5 years ago

  • Focuses ui removed
  • Milestone changed from Awaiting Review to 5.6
  • Summary changed from Html Tag Missing to <ul> tag missing in tests/qunit/index.html

#5 @SergeyBiryukov
5 years ago

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

In 49107:

Build/Test Tools: Add missing <ul> tag in tests/qunit/index.html.

Props ravipatel, JeffPaul.
Fixes #51473.

#6 @SergeyBiryukov
5 years ago

For reference, the section was introduced in [37714].

#7 @SergeyBiryukov
5 years ago

  • Version changed from trunk to 4.6
Note: See TracTickets for help on using tickets.