Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3085)

Unified Diff: chrome/browser/resources/file_manager/css/combobutton.css

Issue 9923005: [WebUI] Fix rest of CSS style nits so I can turn on CSS checker. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: padding fixes, remove a no-op change Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/file_manager/css/combobutton.css
diff --git a/chrome/browser/resources/file_manager/css/combobutton.css b/chrome/browser/resources/file_manager/css/combobutton.css
index 42a225a510cbdd8d5b10d26972fb25a24ac99507..0c9594e90465732d1e31174cfaac3c2d0ab7a322 100644
--- a/chrome/browser/resources/file_manager/css/combobutton.css
+++ b/chrome/browser/resources/file_manager/css/combobutton.css
@@ -1,23 +1,21 @@
-/*
- * Copyright (c) 2012 The Chromium Authors. All rights reserved.
+/* Copyright (c) 2012 The Chromium Authors. All rights reserved.
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
*/
-
.cr-button {
-webkit-appearance: none;
-webkit-user-select: none;
background-image: -webkit-linear-gradient(#ededed, #ededed 38%, #dedede);
- border-radius: 2px;
border: 1px solid rgba(0, 0, 0, 0.25);
+ border-radius: 2px;
box-shadow: 0 1px 0 rgba(0, 0, 0, 0.08),
inset 0 1px 2px rgba(255, 255, 255, 0.75);
color: #444;
+ cursor: default;
font: -webkit-small-control;
margin: 0 1px 0 0;
text-shadow: 0 1px 0 rgb(240, 240, 240);
- cursor: default;
}
.cr-button:hover {
@@ -47,16 +45,16 @@
.cr-combobutton {
- position: relative;
+ cursor: default;
display: none;
font: -webkit-small-control;
- cursor: default;
+ position: relative;
}
.cr-combobutton[visible] {
- display: -webkit-box;
- -webkit-box-orient: vertical;
-webkit-box-align: stretch;
+ -webkit-box-orient: vertical;
+ display: -webkit-box;
}
.cr-cb-container {
@@ -65,12 +63,12 @@
}
.cr-cb-button-container {
+ -webkit-box-align: stretch;
-webkit-box-flex: 1;
- display: -webkit-box;
-webkit-box-orient: vertical;
- -webkit-box-align: stretch;
- border-top-right-radius: 0px;
- border-bottom-right-radius: 0px;
+ border-bottom-right-radius: 0;
+ border-top-right-radius: 0;
+ display: -webkit-box;
}
.cr-cb-button-container > * {
@@ -80,11 +78,11 @@
.cr-cb-trigger {
-webkit-box-flex: 0;
-webkit-margin-start: -1px;
- width: 20px;
+ border-bottom-left-radius: 0;
+ border-top-left-radius: 0;
display: none;
- border-top-left-radius: 0px;
- border-bottom-left-radius: 0px;
margin-left: -2px;
Dan Beam 2012/03/31 07:11:12 I don't think this works in RTL, but I didn't chan
+ width: 20px;
}
.cr-cb-trigger > img {
@@ -98,27 +96,26 @@
}
.cr-cb-popup {
- position: absolute;
- right: 1px;
- display: -webkit-box;
- opacity: 0;
- pointer-events: none;
- -webkit-box-orient: vertical;
-webkit-box-align: stretch;
- box-sizing: border-box;
-
+ -webkit-box-orient: vertical;
-webkit-box-shadow: 0 2px 4px rgba(0, 0, 0, .50);
background: white;
+ border: 1px solid rgba(0, 0, 0, 0.25);
+ border-radius: 2px;
+ box-sizing: border-box;
color: black;
cursor: default;
- white-space: nowrap;
- border-radius: 2px;
- border: 1px solid rgba(0, 0, 0, 0.25);
+ display: -webkit-box;
+ opacity: 0;
padding: 8px 0;
+ pointer-events: none;
+ position: absolute;
+ right: 1px;
+ white-space: nowrap;
}
.cr-cb-popup:active {
- box-shadow: inset 0px 1px 2px rgba(0, 0, 0, 0.1);
+ box-shadow: inset 0 1px 2px rgba(0, 0, 0, 0.1);
}
.cr-combobutton[open] .cr-cb-popup {
@@ -135,10 +132,10 @@
}
.cr-cb-popup > *:hover {
- background-color: #dce5fa;
+ background-color: rgb(220, 229, 250);
}
.cr-cb-popup > *:active {
- background-color: #426dc9;
+ background-color: rgb(66, 109, 201);
color: #fff;
}

Powered by Google App Engine
This is Rietveld 408576698