Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#9429 closed enhancement (fixed)

Output Broken: Theme Widgets Admin Output does not validate

Reported by: hakre Owned by:
Milestone: 2.8 Priority: lowest
Severity: trivial Version: 2.7
Component: Widgets Keywords: has-patch needs-testing
Focuses: Cc:


The HTML Output of widgets.php in the backend does produce a lot of errors. This is mainly because, '%i%' has been choosen as placeholder. But '%' is not a valid char in HTML IDs.

Therefore the only thing to fix this is to change the placeholder to something HTML conform. For Example '.i.'. The attached patch does fix this (and #9428).

Attachments (1)

qcop-r00-admin-widget.patch (4.1 KB) - added by hakre 9 years ago.
patching %i% in widgets handling.

Download all attachments as: .zip

Change History (7)

9 years ago

patching %i% in widgets handling.

#1 follow-up: @hakre
9 years ago

While testing, I run into problems adding new Widgets. Removing does work.

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

Replying to hakre:

While testing, I run into problems adding new Widgets. Removing does work.

my fault, this is related to #9444 which did not load in the .dev.js files. to test the patch, you must enable script debugging for the script-loader.php. To my knowledge this can only be done manually by modifying the script-loader.php file.

#3 @hakre
9 years ago

test done: no problems to add widgets (on wp-admin/widgets.php) with this patch.

#4 @ryan
9 years ago

  • Component changed from General to Widgets
  • Owner anonymous deleted

#5 @Denis-de-Bernardy
9 years ago

  • Keywords needs-testing added
  • Milestone changed from Unassigned to 2.8
  • Priority changed from normal to lowest
  • Severity changed from normal to trivial
  • Type changed from defect (bug) to enhancement
  • Version set to 2.7

better check if this works well with #9511

#6 @ryan
9 years ago

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

We use a different placeholder now, which should address this.

Note: See TracTickets for help on using tickets.