Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#8744 closed enhancement (wontfix)

Few validation fixes for WordPress

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

Description

All changes were tested on my Test Blog.

Changelog

Files:

  • wp-login.php
  • login.css

Validation warning:

  • Same ID fo CSS file LINK and login DIV

Solution:

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

File:

  • dashboard.php

Validation warning:

  • FORM ID and NAME attribute value mismatch

Solution:

  • FORM NAME changed to "quick-press"

File:

  • upload.php

Validation warning:

  • Invalid paragraph content or empty paragraph

Solution:

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

File:

  • tools.php

Validation warning:

  • A tag escaping malformed URI reference

Solution:

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

Attachments (1)

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

Download all attachments as: .zip

Change History (9)

#1 follow-up: @DD32
16 years ago

tools.php

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
16 years ago

Replying to DD32
So why it works for me?

#3 @DD32
16 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
16 years ago

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

#5 @ryan
16 years ago

  • Component changed from General to Validation
  • Owner anonymous deleted

#6 @FFEMTcJ
16 years ago

  • Milestone changed from 2.7.2 to 2.8

Postponing enhancements to 2.8

#8 @ryan
16 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.