Make WordPress Core

Opened 4 months ago

Last modified 2 months ago

#26608 new defect (bug)

/wp-admin/includes/post.php: Make HTML tag standard compliant

Reported by: Latz Owned by:
Milestone: Future Release Priority: lowest
Severity: trivial Version: 3.6
Component: Posts, Post Types Keywords: has-patch 3.9-early
Focuses: administration Cc:


<br> tag isn't closed.

Attachments (1)

26608.diff (695 bytes) - added by Latz 4 months ago.

Download all attachments as: .zip

Change History (8)

Latz4 months ago

comment:1 Latz4 months ago

  • Keywords has-patch added
  • Severity changed from normal to minor

comment:2 SergeyBiryukov4 months ago

  • Keywords 3.9-early added
  • Milestone changed from Awaiting Review to Future Release
  • Priority changed from normal to lowest
  • Severity changed from minor to trivial
  • Version changed from trunk to 3.6

Related: [24042]

comment:3 ocean904 months ago

What do you mean by "HTML tag standard compliant"?

Since we doesn't have a XML document here I'm fine with just <br>. There is really no need to "fix" such things.

See also http://www.w3.org/TR/html5/text-level-semantics.html#the-br-element.

comment:4 Latz4 months ago

You're right, I thought HTML5 would require properly closing all tags. Nevertheless there are the "WordPress Coding standards": Self-closing Elements (http://make.wordpress.org/core/handbook/coding-standards/html/#self-closing-elements). Secondly it's the only br tag in the whole code base that's not self closed. So at least it should be fixed for consistency.

comment:5 follow-up: ocean904 months ago

Secondly it's the only br tag in the whole code base that's not self closed.

There are two more in JS files:

comment:6 in reply to: ↑ 5 Latz4 months ago

Replying to ocean90:

There are two more in JS files:

Does that mean you would support the ticket if those were changed, too?

comment:7 jeremyfelt2 months ago

  • Component changed from Administration to Posts, Post Types
  • Focuses admin added
Note: See TracTickets for help on using tickets.