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

Unified Diff: chrome/browser/resources/chromeos/login/network_dropdown.css

Issue 7982002: [cros,webui] Adds scrollbar for the network drop-down controller. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: added ellipsis for title, fixed width problems Created 9 years, 3 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/chromeos/login/network_dropdown.css
diff --git a/chrome/browser/resources/chromeos/login/network_dropdown.css b/chrome/browser/resources/chromeos/login/network_dropdown.css
index 71d1a910de58cc21736a0edc29b4784524d2ee79..54e3117e3b6d579dd47a5bc196c162451ca0a852 100644
--- a/chrome/browser/resources/chromeos/login/network_dropdown.css
+++ b/chrome/browser/resources/chromeos/login/network_dropdown.css
@@ -42,26 +42,31 @@
}
.dropdown-title img {
- maring-top: -1px;
+ margin-top: -1px;
}
.dropdown-title div {
+ -webkit-box-flex: 1;
margin-top: 1px;
+ overflow: hidden;
+ text-overflow: ellipsis;
}
.dropdown-container {
background: #f7f7f7;
border: 1px solid #7c91bf;
+ max-height: 350px;
Nikita (slow) 2011/09/21 10:44:25 nit: This value should be based on space available
altimofeev 2011/09/21 11:49:53 Use 330px. Done.
+ overflow-x: hidden;
+ overflow-y: auto;
padding-bottom: 2px;
padding-top: 2px;
position: relative;
top: 100%;
- width: 248px;
z-index: 10;
}
.dropdown-item-container {
- -webkitpadding-start: 5px;
+ -webkit-padding-start: 5px;
display: -webkit-box;
height: 24px;
}
@@ -70,18 +75,14 @@
background: #dce4fa;
}
-.dropdown-image {
- width: 24px;
-}
-
.dropdown-item {
+ -webkit-box-flex: 1;
cursor: default;
color: #555;
overflow: hidden;
padding-top: 4px;
text-overflow: ellipsis;
white-space: nowrap;
- width: 219px;
}
.dropdown-divider {

Powered by Google App Engine
This is Rietveld 408576698