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

Unified Diff: chrome/browser/resources/apps_debugger/css/items.css

Issue 15389003: Loosen a bit the HTML hierarchy required for Tabs to work, Improved UX of the dev tools (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove 'mainThis'. Created 7 years, 7 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/apps_debugger/css/items.css
diff --git a/chrome/browser/resources/apps_debugger/css/items.css b/chrome/browser/resources/apps_debugger/css/items.css
index 6b4e990b94a0b6eabdaa170c511589f26165e056..07c417715c504e1dacfb1124c6e02305b84db775 100644
--- a/chrome/browser/resources/apps_debugger/css/items.css
+++ b/chrome/browser/resources/apps_debugger/css/items.css
@@ -17,34 +17,111 @@ hr {
/* Override the visual style of tabs. */
+#tabs-header-container {
+ border-bottom: 1px solid #e0e0e0;
+ height: calc(100% - 53px);
+}
+
+#tabs {
+ border-bottom: none;
+ margin: 5px auto 0;
+ padding: 0 5px;
+ width: 800px;
+}
+
tabs tab {
+ border-color: #e0e0e0;
min-width: 150px;
padding: 5px 20px;
}
-/* Contents */
+tabs > [selected] {
+ background: white;
+ border-color: #e0e0e0;
+}
-#extension-settings {
- margin: 30px;
- max-width: 800px;
+html.focus-outline-visible tabs:focus > [selected] {
+ background: white;
+ border-color: rgb(160, 160, 255);
+ outline: none;
}
-#no-extensions-message,
-#no-apps-message {
- font-weight: bold;
+tabs > :not([selected]) {
+ background: none;
+ border-color: #e0e0e0;
}
-.empty-item-list {
- height: 3em;
+tabs > :not([selected]):hover {
+ background: rgba(255, 255, 255, 0.9);
+}
+
+tabpanels {
+ height: 100%;
+ padding: 0;
+ width: 100%;
+}
+
+tabpanel {
+ margin: 0 auto;
+ padding-top: 80px;
+ width: 800px;
+}
+
+/* Header */
+
+#header {
+ background-color: rgba(255, 255, 255, 0.9);
+ position: fixed;
+ top: 0;
+ width: 100%;
+ z-index: 1;
}
#developer-controls {
- margin-top: 15px;
- max-width: 750px;
+ margin-top: 0;
+ max-width: none;
+ padding: 3px 10px 8px;
+ width: calc(100% - 20px);
+}
+
+#header-title {
+ -webkit-margin-start: 50px;
+ display: inline-block;
+ font-size: 1.2em;
+ position: relative;
+ top: 3px;
+}
+
+#header-bottom-gradient {
+ background: linear-gradient(to bottom, white, rgba(255, 255, 255, 0));
+ height: 30px;
+ position: fixed;
+ top: 79px;
+ width: 100%;
}
#search {
float: right;
+ margin-top: 7px;
+ padding: 4px;
+ width: 200px;
+}
+
+/* Contents */
+
+#extension-settings {
+ margin: 0;
+ max-width: 100%;
+ padding: 10px 0;
+}
+
+#no-extensions-message,
+#no-apps-message {
+ font-weight: bold;
+}
+
+.empty-item-list {
+ height: 3em;
}
html[dir='rtl'] #search {
Dan Beam 2013/05/25 02:43:33 ^ please move this under the #search {} selector
dvh 2013/05/25 03:53:22 Done.

Powered by Google App Engine
This is Rietveld 408576698