Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#8744 closed enhancement (wontfix)

Few validation fixes for WordPress

Reported by: Simek Owned by:
Milestone: Priority: low
Severity: trivial Version: 2.8
Component: Validation Keywords: has-patch
Focuses: Cc:


All changes were tested on my Test Blog.



  • wp-login.php
  • login.css

Validation warning:

  • Same ID fo CSS file LINK and login DIV


  • DIV ID changed from "login" to "loginbox". CSS also corrected.


  • dashboard.php

Validation warning:

  • FORM ID and NAME attribute value mismatch


  • FORM NAME changed to "quick-press"


  • upload.php

Validation warning:

  • Invalid paragraph content or empty paragraph


  • Deleted P tag and P close tag. In this P was DIV with Edit, Delete and View options. P tag was unneeded.


  • tools.php

Validation warning:

  • A tag escaping malformed URI reference


  • changed HREF to "#" and added ONCLICK attribute for JS script.

Attachments (1)

validation_fixes.patch (4.2 KB) - added by Simek 9 years ago.

Download all attachments as: .zip

Change History (9)

#1 follow-up: @DD32
9 years ago


That'll break Press This.

Press This is designed to be used as a bookmarklet, for that to work, the JavaScript needs to be within the href

#2 in reply to: ↑ 1 @Simek
9 years ago

Replying to DD32 So why it works for me?

#3 @DD32
9 years ago

So why it works for me?

Clicking it will, But bookmarking it wont, Nor will draging it onto a menu.

Bookmarks remember a URL, not a onclick method.

#4 @Simek
9 years ago

Publishing also works, but after this explanation I must agree with you.

#5 @ryan
9 years ago

  • Component changed from General to Validation
  • Owner anonymous deleted

9 years ago

  • Milestone changed from 2.7.2 to 2.8

Postponing enhancements to 2.8

#8 @ryan
9 years ago

  • Milestone 2.8 deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Sounds like it. If not, reopen.

Note: See TracTickets for help on using tickets.