WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 8 months ago

#16135 closed defect (bug) (invalid)

Invalid markup in administration pages

Reported by: wahgnube Owned by:
Milestone: Priority: normal
Severity: minor Version: 3.1
Component: Validation Keywords: has-patch
Focuses: Cc:

Description

There still remain (related to #15287) some trivial validation errors in pages in wp-admin. I have attached two tiny patches that fix some of them.

Attachments (2)

update.php.patch (1011 bytes) - added by wahgnube 3 years ago.
themes.php.patch (533 bytes) - added by wahgnube 3 years ago.

Download all attachments as: .zip

Change History (8)

wahgnube3 years ago

wahgnube3 years ago

comment:1 follow-up: nacin3 years ago

Not sure where those amps get passed to. Redirect maybe? Ideally they should be escaped at the last possible point.

The br's can be closed. I thought I caught all of those.

comment:2 nacin3 years ago

Ah, they're already closed. That triggers a validation error?

comment:3 wahgnube3 years ago

Not on a default WordPress install with html_type set to text/html. A few years ago I set it to application/xhtml+xml and I've been running into tiny things like this on wp-admin ever since. (Because of wp-admin/admin-header.php
does:
@header('Content-Type: ' . get_option('html_type') . '; charset=' . get_option('blog_charset')); )


Version 0, edited 3 years ago by wahgnube (next)

comment:4 in reply to: ↑ 1 scribu3 years ago

Replying to nacin:

The br's can be closed. I thought I caught all of those.

Crokford say:

But I see people get really exercised ? about that. There's a devotion to web standards that is completely unjustified, and this is just one example, a trivial example. It's as if:

"and the Gods gave us the Web Standards, and deviation from the Web Standards is the source of All Evil!"

It turns out there's no truth in that statement at all. I know the guys who made the web standards, and there were some really bright guys in those numbers and there was some other guys who were not nearly that bright. There was a lot of time pressure, there was a lot of mismanagement, a lot of mistakes were made. Given the process that put it all together, it's amazing that it's as good as it is. But it's way far from perfect, and it's way less than we deserve.

http://developer.yahoo.com/yui/theater/video.php?v=crockonjs-4

comment:5 markjaquith3 years ago

  • Milestone changed from Awaiting Review to Future Release

comment:6 dd328 months ago

  • Milestone Future Release deleted
  • Resolution set to invalid
  • Status changed from new to closed

Neither of these two cases apply anymore.

Note: See TracTickets for help on using tickets.