Update thumbnail/list view buttons.

chrome/browser/resources/file_manager/css/file_manager.css

Index: chrome/browser/resources/file_manager/css/file_manager.css
diff --git a/chrome/browser/resources/file_manager/css/file_manager.css b/chrome/browser/resources/file_manager/css/file_manager.css
index 2e5074dab3e4a7f05da41bcb7e79289b3888cea5..8ef619e25fd6e29d742452868aa51fca1a8be328 100644
--- a/chrome/browser/resources/file_manager/css/file_manager.css
+++ b/chrome/browser/resources/file_manager/css/file_manager.css
@@ -330,10 +330,35 @@ div.open-sidebar:hover {
   font-size: 12px;
 }
 
-button.detail-view > img,
-button.thumbnail-view > img {
-  position: relative;
-  top: 1px;
+button.detail-view,
+button.thumbnail-view {

[flackr, 2012/01/10 22:02:55]

If you can remove the outer div, change this to .dialog-header > button {

[bshe, 2012/01/11 01:16:21]

outer div is a hack for extra margin between buttons. Use .dialog-header > div >
button instead.

+  -webkit-border-radius: 0;
+  border: none;
+  height: 28px;
+  margin: 0;
+  min-width: 0;

[flackr, 2012/01/10 22:02:55]

Did you mean to specify this as well as width below?

[bshe, 2012/01/11 01:16:21]

Yes. I need it to override this:
button,
input[type='button'],
input[type='submit'] {
  min-width: 4em;
}
It seems 28px is smaller than 4em in this case.

+  padding: 0;
+  width: 28px;
+}
+
+button.detail-view,
+button.detail-view:hover {

[flackr, 2012/01/10 22:02:55]

Seems like you shouldn't need a selector for the hover states as well. Can you
remove these?

[bshe, 2012/01/11 01:16:21]

The hover states aren't necessary only when these background-img rules appears
after these rules in css file:
button:hover {
  background: #ebebeb -webkit-linear-gradient(#fefefe, #f8f8f8 40%, #e9e9e9);
}
and
button[disabled]:hover {
  background: -webkit-linear-gradient(#fafafa, #f4f4f4 40%, #e5e5e5);
}

I can either add a comment here to say do not change the order of these rules or
keep the hover state here just in case someone move these rules and causing
problem. Which way do you suggest?

[flackr, 2012/01/11 16:25:45]

I suggest a comment to the effect that you must override the defined
button:hover and button[disabled]:hover rules.

+  background-image: url('../images/icon_list_view_off.png');
+}
+
+button.thumbnail-view,
+button.thumbnail-view:hover {
+  background-image: url('../images/icon_thumb_view_off.png');
+}
+
+button.detail-view[disabled],
+button.detail-view[disabled]:hover {
+  background-image: url('../images/icon_list_view_on.png');
+}
+
+button.thumbnail-view[disabled],
+button.thumbnail-view[disabled]:hover {
+  background-image: url('../images/icon_thumb_view_on.png');
 }
 
 .filelist-panel {