Opened 11 years ago
Closed 11 years ago
#24293 closed enhancement (fixed)
$allowedposttags to allow value for <li>
Reported by: | kraftbj | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Formatting | Keywords: | has-patch commit 3.7-early |
Focuses: | Cc: |
Description
HTML 5 allowed the value
attribute in <li>
elements for use with ol
s (e.g. setting <ol><li value="31">
would result in that item of the ol
rendering as 31.
$allowedposttags strips that out currently, as that's not allowed.
Submitting a patch to allow the value attribute.
Attachments (1)
Change History (7)
#1
@
11 years ago
- Cc bk@… added
- Keywords has-patch added
Info regarding value attr: https://developer.mozilla.org/en-US/docs/HTML/Element/li
#3
@
11 years ago
From the last IRC chat, nacin mentioned this may not be so straightforward of a patch, needing to ensure proper escaping, etc. I'm not sure the best way to ensure of this since we're just needing to add an extra attribute.
Happy to do the work to learn the process; can someone help me with what direction to go in to ensure that adding value attribute to the li tag works as it should?
A very similar "start" attribute to the ol tag is already in $allowedposttags.
added value to li in $allowedposttags in /wp-includes/kses.php