[wp-trac] [WordPress Trac] #47178: Twenty Ten: Remove an extra colons from blocks.css

WordPress Trac noreply at wordpress.org
Fri May 17 16:08:33 UTC 2019


#47178: Twenty Ten: Remove an extra colons from blocks.css
-------------------------------------+-------------------------------
 Reporter:  shashank3105             |       Owner:  (none)
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  5.3
Component:  Bundled Theme            |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-refresh  |     Focuses:  coding-standards
-------------------------------------+-------------------------------
Changes (by desrosj):

 * keywords:  has-patch commit => has-patch needs-refresh


Comment:

 I did some digging into the `rtl.css` file in Twenty Sixteen with the
 double `;` and it appears that this file has not been updated since the
 theme was moved over to Core's SVN repo. It was also not updated after
 being added to the [https://github.com/wordpress/twentysixteen GitHub
 repository].

 It appears that RTL stylesheets are not automatically generated from the
 LTR versions in bundled themes prior to Twenty Nineteen (which has its
 [https://core.trac.wordpress.org/browser/trunk/src/wp-
 content/themes/twentynineteen/package.json#L39 own build process]).

 Because these are not automatically generated by a build process, it
 should be safe to remove the double `;` in `twentysixteen/rtl.css` as
 well.

 @laurelfulford Since you have a bit more historic knowledge of the bundled
 themes, can you confirm?

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


More information about the wp-trac mailing list