Index: chrome/browser/resources/net_internals/main.css |
diff --git a/chrome/browser/resources/net_internals/main.css b/chrome/browser/resources/net_internals/main.css |
index 655e27baeca6da061f6a2973b5672bfe82f3deeb..2905cbfc8e9fe8694a3758b33791eefc99951832 100644 |
--- a/chrome/browser/resources/net_internals/main.css |
+++ b/chrome/browser/resources/net_internals/main.css |
@@ -1,8 +1,8 @@ |
/* |
mmenke
2012/02/28 22:15:02
Should we be putting a second asterisk on these?
eroman
2012/02/28 22:28:32
We have some inconsistencies. I'll do another pass
|
-Copyright (c) 2011 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. |
-*/ |
+ * Copyright (c) 2011 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. |
+ */ |
/* Only common styles should be declared here. */ |
@@ -26,38 +26,22 @@ body { |
user-select: none; |
} |
-.logSourceEntry { |
+.log-source-entry { |
margin: 5px; |
} |
-.logSourceEntry * p { |
+.log-source-entry * p { |
font-weight: bold; |
font-size: 12px; |
} |
-.logSourceEntry * td { |
+.log-source-entry * td { |
font-size: 10px; |
} |
-/* TODO(eroman): make this a classname instead */ |
-#events-view-details-log-box, |
-#events-view-details-timeline-box, |
-#http-cache-view-tab-content, |
-#proxy-view-tab-content, |
-#capture-view-tab-content, |
-#import-view-tab-content, |
-#export-view-tab-content, |
-#dns-view-tab-content, |
-#sockets-view-tab-content, |
-#spdy-view-tab-content, |
-#service-providers-view-tab-content, |
-#test-view-tab-content, |
-#hsts-view-tab-content, |
-#http-throttling-view-tab-content, |
-#logs-view-tab-content, |
-#prerender-view-tab-content, |
-#http-pipeline-view-tab-content, |
-#chromeos-view-tab-content { |
+/* This class should be given to top-level content boxes (like the view's main |
+ * DIV). It gives them a consistent margin, and makes them scrollable. */ |
mmenke
2012/02/28 22:15:02
nit: Should make this consistent with the other c
eroman
2012/02/28 22:28:32
Done.
However we use both styles in this file.
|
+.content-box { |
mmenke
2012/02/28 22:15:02
Much better.
|
overflow: auto; |
padding: 10px; |
} |
@@ -65,23 +49,23 @@ body { |
/* |
* Styles for TABLE that uses a thin collapsed border. |
*/ |
-table.styledTable { |
+table.styled-table { |
border-collapse: collapse; |
} |
-table.styledTable, |
-.styledTable th, |
-.styledTable td { |
+table.styled-table, |
+.styled-table th, |
+.styled-table td { |
mmenke
2012/02/28 22:15:02
The unit tests use the name of this class to make
eroman
2012/02/28 22:28:32
Done.
|
border: 1px solid #777; |
padding-right: 4px; |
padding-left: 4px; |
} |
-.styledTable th { |
+.styled-table th { |
background: rgb(224,236,255); |
} |
-.styledTable th.title { |
+.styled-table th.title { |
background: rgb(255,217,217); |
} |
@@ -102,16 +86,8 @@ table.styledTable, |
} |
/** |
- * Styling for an emphasized button. |
- */ |
-.bigButton { |
- font-size: 100%; |
- font-weight: bold; |
-} |
- |
-/** |
* Styling for text indicating a potential problem or error state. |
*/ |
-.warningText { |
+.warning-text { |
color: red; |
} |