[wp-trac] [WordPress Trac] #39087: Customizer: Add indicator when additional widget areas are hidden

WordPress Trac noreply at wordpress.org
Wed Mar 22 20:20:51 UTC 2017


#39087: Customizer: Add indicator when additional widget areas are hidden
-------------------------------------------------+-------------------------
 Reporter:  shireling                            |       Owner:
     Type:  enhancement                          |  westonruter
 Priority:  normal                               |      Status:  reopened
Component:  Customize                            |   Milestone:  4.7.4
 Severity:  normal                               |     Version:  3.9
 Keywords:  has-ux-feedback has-patch needs-     |  Resolution:
  testing has-screenshots i18n-change            |     Focuses:
-------------------------------------------------+-------------------------

Comment (by westonruter):

 @ocean90 thanks for the feedback. You're absolutely right about the
 translation problem. How does [attachment:39087.6.diff] look for a
 workaround?

 As for the strings not using curly quotes, these strings don't technically
 have quotation marks but rather have apostrophes for the contractions, so
 `APOSTROPHE (U+0027)` seems more appropriate. That being said, if we were
 to use non-ASCII characters in these strings, there is also the issues
 encountered in #32875 with the difficulties encoding unicode characters in
 JSON which resulted in [34087] [34233]. Is there an updated best practice
 for that? The workaround would be to just go ahead and use HTML entities,
 but then this would mean `p.html( string )` would have to be used instead
 of `p.text( string )`, or likewise `{{{...}}}` in templates instead of
 `{{...}}`. I know that's not frowned upon in core, but normally I'd flag
 this as an opportunity for an injection vulnerability.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/39087#comment:34>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list