Opened 8 years ago
Closed 8 years ago
#39728 closed defect (bug) (duplicate)
"Additional CSS" validate syntax error in customizer
Reported by: | rishishah | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.8 |
Component: | Customize | Keywords: | |
Focuses: | ui | Cc: |
Description
Validation for Additional CSS in customize is not correct.
STEP TO REPRODUCE
- Go to "Appearance>Customize"
- Select "Additional CSS"
- When I wrote curly brace "{" or "}" inside comment section /* ... */ and save it, it shows me error that
Your curly brackets {} are imbalanced. Make sure there is a closing } for every opening {.
This error should not consider inside comment section /* ... */
Attachments (1)
Change History (4)
Note: See
TracTickets for help on using
tickets.
See also #39198 for where we removed the check for single quotes
'
but left it in place for{
,}
, and"
.We've also got #39218 to use a tokeniser rather than naive regular expressions.
I don't think we should change it further for the 4.7.x branch myself (unless we backport tokenisation). I'd suggest marking this as a duplicate of #39218.