Clean up css in simpletest module [#1663164]

Sub-issue of #1190252: [573] Use csslint as a weapon to beat the crappy CSS out of Drupal core

Inline with the CSS cleanup efforts of the HTML5 initiative, using CSSLint at http://csslint.net provides a quick way to code-sniff our css and tweak styles.

Copy and paste the the stylesheet(s) below into the css lint tool at http://csslint.net and test.
Fix any warnings or errors the tool finds.
Patch Drupal 8 locally and make sure the css changes have not broken anything visually.
Create patch and upload for the testbot.

Files: modules/simpletest/simpletest.css

Comment	File	Size	Author
#1	1663164-simpletest-css-cleanup.patch	2.68 KB	droplet

Comments

Comment #1

droplet commented 29 June 2012 at 03:29

Status:

Active

» Needs work

Status	File	Size
new	1663164-simpletest-css-cleanup.patch	2.68 KB

This is very bad and I can't find any usage example.

@@ -39,46 +39,46 @@ div.message > div.item-list {
   font-weight: normal;
 }

Comment #2

droplet commented 29 June 2012 at 03:33

+++ b/core/modules/simpletest/simpletest.cssundefined
@@ -39,46 +39,46 @@ div.message > div.item-list {
-tr.simpletest-pass.odd {
+.simpletest-pass.odd {
   background-color: #b6ffb6;
 }
-tr.simpletest-pass.even {
+.simpletest-pass.even {
   background-color: #9bff9b;
 }
-tr.simpletest-fail.odd {
+.simpletest-fail.odd {
   background-color: #ffc9c9;
 }
-tr.simpletest-fail.even {
+.simpletest-fail.even {
   background-color: #ffacac;
 }
-tr.simpletest-exception.odd {
+.simpletest-exception.odd {
   background-color: #f4ea71;
 }
-tr.simpletest-exception.even {
+.simpletest-exception.even {
   background-color: #f5e742;
 }
-tr.simpletest-debug.odd {
+.simpletest-debug.odd {
   background-color: #eee;
 }
-tr.simpletest-debug.even {
+.simpletest-debug.even {
   background-color: #fff;
 }

http://drupal.org/node/1649780