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

Unified 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, 2 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/password_manager.css
diff --git a/chrome/browser/resources/options/password_manager.css b/chrome/browser/resources/options/password_manager.css
index 18314ef98e861084f8d5fffa15d8ec551eb9b53b..a24236ccbae4aabeb1dbb51483fdf15c27944db7 100644
--- a/chrome/browser/resources/options/password_manager.css
+++ b/chrome/browser/resources/options/password_manager.css
@@ -26,3 +26,12 @@ html[dir=rtl] #password-search-column {
#passwords-title {
display: inline-block;
}
+
+#manage-passwords-link {
+ 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
+}
+
+#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.
+ bottom: 18px;
+ position: absolute;
+}

Powered by Google App Engine
This is Rietveld 408576698