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

Side by Side Diff: chrome/browser/resources/options/password_manager.css

Issue 682133003: [Password Manager] Add link to remotely manage passowrds on desktop (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup Created 6 years, 1 month 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 5
6 #password-manager > div.content-area { 6 #password-manager > div.content-area {
7 width: 600px; 7 width: 600px;
8 } 8 }
9 9
10 #password-search-column { 10 #password-search-column {
11 bottom: 10px; 11 bottom: 10px;
12 position: absolute; 12 position: absolute;
13 right: 0; 13 right: 0;
14 } 14 }
15 15
16 html[dir=rtl] #password-search-column { 16 html[dir=rtl] #password-search-column {
17 left: 0; 17 left: 0;
18 right: auto; 18 right: auto;
19 } 19 }
20 20
21 #password-list-headers { 21 #password-list-headers {
22 position: relative; 22 position: relative;
23 width: 100%; 23 width: 100%;
24 } 24 }
25 25
26 #passwords-title { 26 #passwords-title {
27 display: inline-block; 27 display: inline-block;
28 } 28 }
29
30 #manage-passwords-link {
31 text-decoration: none;
Evan Stade 2014/10/28 22:50:48 pretty sure this rule should not be here. Links to
Garrett Casto 2014/10/28 23:18:55 Okay. Mocks had this without the underline and I s
32 }
33
34 #manage-passwords-span {
Evan Stade 2014/10/28 22:50:48 don't think you need this either. The span should
Garrett Casto 2014/10/28 23:18:55 To be clear, are you saying that this isn't necess
Evan Stade 2014/10/28 23:26:01 both places should add a spacer div right after th
Garrett Casto 2014/10/29 20:25:46 Sounds good. Unfortunately this fails in PRESUBMIT
Evan Stade 2014/10/29 21:28:28 I think it's ok to ignore this PRESUBMIT check.
35 bottom: 18px;
36 position: absolute;
37 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698