Opened 6 years ago
Last modified 7 weeks ago
#47976 reopened defect (bug)
HTML attributes in uppercase show error in widgets
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | normal | Version: | 4.9 |
| Component: | Widgets | Keywords: | has-screenshots dev-feedback has-patch |
| Focuses: | administration | Cc: |
Description
(1) How to reproduce
go to p-admin/widgets.php and add custom HTML.
in custom html box if i add this html, it will error.
<div id="abcd" DATA-HREF="xerjss"></div>
(2) Error - the attribute name of [ "DATA-HREF" ] must be in lowercase.
(3) Why need this
some ad providers (such as flipkart) supply code where attributes are uppercase.
Attachments (2)
Change History (10)
#3
@
5 years ago
- Version set to 5.5.2
Can anyone take a look at this? Even I am having the same problem as @FlorianBrinkmann with using SVG code.
#4
@
5 years ago
- Type changed from enhancement to defect (bug)
Same issue here. Cannot save an HTML widget with attribute "viewBox" (WP 5.7).
Error: The attribute name of [ viewBox ] must be in lowercase.
#5
@
5 years ago
- Keywords has-patch added
- Version changed from 5.5.2 to 4.9
47976.diff simply disables the error for uppercase letters in attributes.
If the error is important to keep for most situations, the attr-lowercase rule could have an array of specific exceptions instead:
['viewBox', 'DATA-HREF']
#6
@
3 years ago
- Resolution set to invalid
- Status changed from new to closed
This is issue is still there when you use the Classic Widgets (https://wordpress.org/plugins/classic-widgets/) plugin.
#8
@
7 weeks ago
47976.diff still applies to trunk, though removing 'attr-lowercase' => true, seems to work just as well as setting it to false.
HTMLHint 1.1.0 modified the rule to ignore camelCase attributes found in SVG elements. #42780 might update HTMLHint.

I am having the same issue: I tried to add an SVG with the
viewBoxattribute, but am not able to save it because of the »error«.