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

Side by Side Diff: chrome/browser/resources/shared/css/chrome_shared2_touch.css

Issue 10222007: Switch from margin to padding for checkboxes and radio buttons. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove margin for checkbox and radio button. Created 8 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 /* NB: do not include this file directly. Include chrome_shared2.css. If touch 5 /* NB: do not include this file directly. Include chrome_shared2.css. If touch
6 * optimization is enabled, these rules will be automatically loaded by the 6 * optimization is enabled, these rules will be automatically loaded by the
7 * SharedResourceDataSource. */ 7 * SharedResourceDataSource. */
8 8
9 <include src="chrome_shared2.css"> 9 <include src="chrome_shared2.css">
10 10
11 /* TODO(kevers): Add [type='input'] and [type='password']. */ 11 /* TODO(kevers): Add [type='input'] and [type='password']. */
12 input[type='search'] { 12 input[type='search'] {
13 padding-bottom: 6px; 13 padding-bottom: 6px;
14 padding-top: 6px; 14 padding-top: 6px;
15 } 15 }
16 16
17 /* Additional padding to disambiguate controls on neighboring rows. 17 /* Margins are not touchable. Use padding on the inner label instead to
18 * Ideally, the margin belongs on input[type=radio|checkbox] to be a more 18 increase the size of the touch target. */
19 * general fix; however, this creates baseline problems as webkit is not
20 * currently able to reconcile the baseline of a text-free inline box
21 * containing a margin and fixed dimensions with neighboring content.
22 * TODO(kevers): Reinvestigate once the baseline adjustment properties in
23 * CSS3 are supported. */
24 .checkbox, 19 .checkbox,
25 .radio { 20 .radio {
26 margin: 12px 0; 21 margin: 0;
22 }
23
24 .checkbox label,
25 .radio label {
26 padding-bottom: 7px;
27 padding-top: 7px;
27 } 28 }
28 29
29 li > button.link-button { 30 li > button.link-button {
30 padding-bottom: 8px; 31 padding-bottom: 8px;
31 padding-top: 8px; 32 padding-top: 8px;
32 } 33 }
33 34
34 button:not(.custom-appearance):not(.link-button), 35 button:not(.custom-appearance):not(.link-button),
35 input[type=button]:not(.custom-appearance), 36 input[type=button]:not(.custom-appearance),
36 input[type=submit]:not(.custom-appearance), 37 input[type=submit]:not(.custom-appearance),
37 select { 38 select {
38 margin-bottom: 6px; 39 margin-bottom: 6px;
39 margin-top: 6px; 40 margin-top: 6px;
40 } 41 }
41 42
42 html[touch-optimized] .tree-item > .tree-row { 43 html[touch-optimized] .tree-item > .tree-row {
43 padding: 4px 3px; 44 padding: 4px 3px;
44 } 45 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698