[wp-trac] [WordPress Trac] #21152: Kill use of colons in class names

WordPress Trac wp-trac at lists.automattic.com
Wed Jul 4 05:25:28 UTC 2012


#21152: Kill use of colons in class names
----------------------------+-----------------------------
 Reporter:  markjaquith     |      Owner:
     Type:  defect (bug)    |     Status:  new
 Priority:  normal          |  Milestone:  Awaiting Review
Component:  Administration  |    Version:  3.4
 Severity:  normal          |   Keywords:
----------------------------+-----------------------------
 Some of the WordPress admin uses class names with colon-separated values
 that have special meanings in our JS code This is a bad idea. Colons are a
 special character in CSS selector engines.

 See #21106 for a really odd bug that cropped up because a css class
 contained the text {{{foo:event}}} which got parsed as an {{{:even}}}
 pseudo-class in JS.

 HTML5 data attributes are the better way forward, in terms of attaching
 data to dom elements. We should kill all instances of colons in favor of
 data attributes.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21152>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list