Changeset 27107

Timestamp:

02/06/2014 07:07:42 AM (7 years ago)

Author:

helen

Message:

Some tweaks to [27106]:

• Add more @todo markings and remove one that's been resolved.
• Restore a missing comma in a .wp-badge background rule, and add a comment reminding us why it's there.

see #18380.

File:

• trunk/src/wp-admin/css/wp-admin.css (modified) (5 diffs)

trunk/src/wp-admin/css/wp-admin.css

@@ -244 +244 @@
 }
 
-/* Large chunks of shared color, from the colors.css merge */
+/* @todo: Take a second look. Large chunks of shared color, from the colors.css merge */
 .widget-top,
 .menu-item-handle,
@@ -4148 +4148 @@
 }
 
-/* unclear if the following tablenav rules are actually used.
+/* @todo: unclear if the following tablenav rules are actually used.
 classes exist in paginate_links() but not seen in list table output. */
 .tablenav .dots {
@@ -4625 +4625 @@
 }
 
-/* @todo: do we really need this? */
+/* @todo: do we really need this? word on the street is we don't and this
+stray rule may actually be compensated for elsewhere. */
 #normal-sortables .submitbox .submitdelete:hover {
     color: #000;
@@ -8727 +8728 @@
 }
 
-/* @todo: what is this for? */
 .about-wrap .changelog.point-releases h3:first-child {
     padding-top: 7px;
@@ -8736 +8736 @@
 .wp-badge {
     background: #0074a2 url('../images/w-logo-white.png?ver=20131202') no-repeat;
-    background: #0074a2 url('../images/wordpress-logo-white.svg?ver=20131110') no-repeat;
+    background: #0074a2, url('../images/wordpress-logo-white.svg?ver=20131110') no-repeat; /* multiple backgrounds are ignored by browsers that don't support SVGs */
     background-position: center 24px;
     background-size: 85px 85px;