chrome/browser/resources/shared/css/bubble_button.css - Issue 11864019: Add base class cr.ui.BubbleButton for options.ControlledSettingIndicator

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 11864019: Add base class cr.ui.BubbleButton for options.ControlledSettingIndicator (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 7 years, 11 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

OLD	NEW
(Empty)
	1 /* Copyright (c) 2013 The Chromium Authors. All rights reserved.

	2 * Use of this source code is governed by a BSD-style license that can be

	3 * found in the LICENSE file. */
	James Hawkins 2013/01/11 18:49:22 nit: Final / should be on the next line. nit: Final / should be on the next line. bartfab (slow) 2013/01/11 19:45:56 Done. Interestingly, all other CSS and JS files I Show quoted text On 2013/01/11 18:49:22, James Hawkins wrote: > nit: Final / should be on the next line. Done. Interestingly, all other CSS and JS files I checked also had the trailing / in their license headers. James Hawkins 2013/01/11 19:48:47 I step away for 8 months and... Show quoted text On 2013/01/11 19:45:56, bartfab wrote: > On 2013/01/11 18:49:22, James Hawkins wrote: > > nit: Final / should be on the next line. > > Done. Interestingly, all other CSS and JS files I checked also had the trailing > / in their license headers. I step away for 8 months and... Dan Beam 2013/01/11 20:32:11 This is not specified anywhere and the last I reme Show quoted text On 2013/01/11 19:45:56, bartfab wrote: > On 2013/01/11 18:49:22, James Hawkins wrote: > > nit: Final / should be on the next line. > > Done. Interestingly, all other CSS and JS files I checked also had the trailing > / in their license headers. This is not specified anywhere and the last I remember we weren't suggesting either way, right?
	4

	5 .bubble-button {

	6 display: inline-block;

	7 height: 0;

	8 line-height: 1;

	9 vertical-align: middle;

	10 width: 16px;

	11 }

	12

	13 .bubble-button > div {

	14 background-size: 16px;

	15 height: 16px;

	16 position: relative;

	17 top: -8px;

	18 width: 16px;

	19 }

OLD	NEW

« no previous file with comments | « chrome/browser/resources/options/options_page.css ('k') | chrome/browser/resources/shared/js/cr/ui/bubble_button.js » ('j') | no next file with comments »