Opened 17 years ago
Closed 17 years ago
#9429 closed enhancement (fixed)
Output Broken: Theme Widgets Admin Output does not validate
| Reported by: | | Owned by: | |
|---|---|---|---|
| Milestone: | 2.8 | Priority: | lowest |
| Severity: | trivial | Version: | 2.7 |
| Component: | Widgets | Keywords: | has-patch needs-testing |
| Focuses: | Cc: |
Description
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)
Change History (7)
#1 follow-up: ↓ 2
@
17 years ago
While testing, I run into problems adding new Widgets. Removing does work.
#2 in reply to: ↑ 1
@
17 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.
patching %i% in widgets handling.