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

Side by Side Diff: chrome/browser/resources/performance_monitor/chart.css

Issue 10820031: Modifications to performance monitor UI to address real webui. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Revision with all latest changes Created 8 years, 4 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 unified diff | Download patch
OLDNEW
1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 * Use of this source code is governed by a BSD-style license that can be 2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file. */ 3 * found in the LICENSE file. */
4 4
5 div#chooseMetrics { 5 #chooseBlock {
Dan Beam 2012/08/03 18:40:36 the style guide says this should be #choose-block
clintstaley 2012/08/06 21:06:47 Sorry. Changed here and elsewhere.
6 float: left; 6 -webkit-box-orient: horizontal;
7 background: #eee;
8 display: -webkit-box;
7 } 9 }
8 10
9 div#chooseEvents { 11 .spacer {
10 float: left; 12 -webkit-box-flex: 1;
11 margin-left: 20px;
12 } 13 }
13 14
14 div#chooseTimeRange { 15 /* Style for div enclosing all charts, collectively. Increase these
15 float: right; 16 * dimensions if you redesign for more than one chart.*/
Dan Beam 2012/08/03 18:40:36 some people prefer not to use "we" or "you" in com
clintstaley 2012/08/06 21:06:47 Can change it if you really feel strongly, but my
Evan Stade 2012/08/07 00:46:35 I don't care about the use of pronouns but it's pr
17 #charts {
18 height: 500px;
19 width: 900px;
16 } 20 }
17 21
18 div#chooseBlock { 22 /* Style for any div that encloses a single chart -- a child div of the
19 background: #eee; 23 * larger #charts div.*/
20 overflow: hidden; 24 .chart {
Dan Beam 2012/08/03 18:40:36 why can't these two rules be combined until they d
clintstaley 2012/08/06 21:06:47 Done.
21 } 25 height: 500px;
22 26 width: 900px;
23 div#charts {
24 height: 500px;
25 width: 900px;
26 }
27
28 div.chart {
29 height: 500px;
30 width: 900px;
31 } 27 }
32 28
33 div.event-label { 29 div.event-label {
34 background: #fff; 30 background: white;
35 border: 1px solid; 31 border: 1px solid;
36 font: 0.6em; 32 font: 0.6em;
37 position: absolute; 33 position: absolute;
38 } 34 }
39
40 .hidden {
41 visibility: hidden;
42 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698