Bug 392802 - "File listings very hard to read" [p=dao@mozilla.com (D��o Gottwald) r=Mano sr=bzbarsky ui-r=beltzner a=blocking-firefox3+]

netwerk/streamconv/converters/nsIndexedToHTML.cpp

@@ -289,7 +289,7 @@ nsIndexedToHTML::OnStartRequest(nsIRequest* request, nsISupports *aContext) {
                          "table[order] > thead > tr > th::after {\n"
                          "  display: none;\n"
                          "  width: .8em;\n"
-                         "  margin-right: -.8em;\n"
+                         "  -moz-margin-end: -.8em;\n"
                          "  text-align: right;\n"
                          "}\n"
                          "table[order=\"asc\"] > thead > tr > th::after {\n"
@@ -314,19 +314,28 @@ nsIndexedToHTML::OnStartRequest(nsIRequest* request, nsISupports *aContext) {
                          "td > a {\n"
                          "  display: block;\n"
                          "}\n"
+                         "/* name */\n"
+                         "th:first-child {\n"
+                         "  -moz-padding-end: 2em;\n"
+                         "}\n"
                          "/* size */\n"
+                         "th:first-child + th {\n"
+                         "  -moz-padding-end: 1em;\n"
+                         "}\n"
                          "td:first-child + td {\n"
                          "  text-align: right;\n"
-                         "  padding-right: 2em;\n"
+                         "  -moz-padding-end: 1em;\n"
                          "  white-space: nowrap;\n"
                          "}\n"
                          "/* date */\n"
                          "td:first-child + td + td {\n"
-                         "  padding-right: 1em;\n"
+                         "  -moz-padding-start: 1em;\n"
+                         "  -moz-padding-end: .5em;\n"
                          "  white-space: nowrap;\n"
                          "}\n"
                          "/* time */\n"
                          "td:last-child {\n"
+                         "  -moz-padding-start: .5em;\n"
                          "  white-space: nowrap;\n"
                          "}\n"
                          "@-moz-document url-prefix(gopher://) {\n"
@@ -341,13 +350,12 @@ nsIndexedToHTML::OnStartRequest(nsIRequest* request, nsISupports *aContext) {
                          ".dir ,\n"
                          ".symlink ,\n"
                          ".file {\n"
-                         "  padding: 0 .5em;\n"
-                         "  margin-left: 20px;\n"
+                         "  -moz-margin-start: 20px;\n"
                          "}\n"
                          ".dir::before ,\n"
                          ".file > img {\n"
-                         "  margin-right: 4px;\n"
-                         "  margin-left: -20px;\n"
+                         "  -moz-margin-end: 4px;\n"
+                         "  -moz-margin-start: -20px;\n"
                          "  vertical-align: middle;\n"
                          "}\n"
                          ".dir::before {\n"

toolkit/themes/pinstripe/global/dirListing/dirListing.css

@@ -82,10 +82,6 @@ p {
   float: right;
 }
 
-#UI_showHidden:not(:hover) {
-  color: graytext;
-}
-
 table {
   clear: both;
   width: 90%;
@@ -96,21 +92,18 @@ thead {
   font-size: 130%;
 }
 
-th {
-  font-weight: normal;
-}
-
-th:first-child ,
-th:first-child + th {
-  padding-right: 2em;
+/* last modified */
+th:last-child {
+  text-align: center;
 }
 
 table[order] > thead > tr > th:hover {
   text-decoration: underline;
 }
 
-tbody > tr:not(:hover) > td:not(:first-child) {
-  color: graytext;
+tbody > tr:hover {
+  outline: 1px solid ThreeDLightShadow;
+  -moz-outline-radius: .3em;
 }
 
 /* let 'Size' and 'Last Modified' take only as much space as they need and 'Name' all the rest */
@@ -118,23 +111,15 @@ td:not(:first-child) {
   width: 0;
 }
 
-.up ,
-.dir ,
-.symlink ,
-.file {
+.up {
   padding: 0 .5em;
-  margin-left: 20px;
-}
-
-.up::before ,
-.dir::before ,
-.file > img {
-  margin-right: 4px;
-  margin-left: -20px;
-  vertical-align: middle;
+  -moz-margin-start: 20px;
 }
 
 .up::before {
+  -moz-margin-end: 4px;
+  -moz-margin-start: -20px;
+  vertical-align: middle;
   content: url(chrome://global/skin/dirListing/up.png);
 }
 

toolkit/themes/winstripe/global/dirListing/dirListing.css

@@ -82,10 +82,6 @@ p {
   float: right;
 }
 
-#UI_showHidden:not(:hover) {
-  color: graytext;
-}
-
 table {
   clear: both;
   width: 90%;
@@ -96,21 +92,18 @@ thead {
   font-size: 130%;
 }
 
-th {
-  font-weight: normal;
-}
-
-th:first-child ,
-th:first-child + th {
-  padding-right: 2em;
+/* last modified */
+th:last-child {
+  text-align: center;
 }
 
 table[order] > thead > tr > th:hover {
   text-decoration: underline;
 }
 
-tbody > tr:not(:hover) > td:not(:first-child) {
-  color: graytext;
+tbody > tr:hover {
+  outline: 1px solid ThreeDLightShadow;
+  -moz-outline-radius: .3em;
 }
 
 /* let 'Size' and 'Last Modified' take only as much space as they need and 'Name' all the rest */
@@ -118,32 +111,22 @@ td:not(:first-child) {
   width: 0;
 }
 
-.up ,
-.dir ,
-.symlink ,
-.file {
+.up {
   padding: 0 .5em;
-  margin-left: 20px;
-}
-
-.up::before ,
-.dir::before ,
-.file > img {
-  margin-right: 4px;
-  margin-left: -20px;
-  vertical-align: middle;
-}
-
-%ifdef MOZ_WIDGET_GTK2
-.dir::before {
-  content: url(moz-icon://stock/gtk-directory?size=menu);
+  -moz-margin-start: 20px;
 }
 
 .up::before {
+  -moz-margin-end: 4px;
+  -moz-margin-start: -20px;
+  vertical-align: middle;
+%ifdef MOZ_WIDGET_GTK2
   content: url(moz-icon://stock/gtk-go-up?size=menu);
 }
+.dir::before {
+  content: url(moz-icon://stock/gtk-directory?size=menu);
 %else
-.up::before {
   content: url(chrome://global/skin/dirListing/up.png);
-}
 %endif
+}
+