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

Unified Diff: chrome/browser/resources/options/clear_browser_data_overlay.css

Issue 1813023002: Add a notice about other forms of history to the CBD dialog (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@add_component
Patch Set: Created 4 years, 9 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/clear_browser_data_overlay.css
diff --git a/chrome/browser/resources/options/clear_browser_data_overlay.css b/chrome/browser/resources/options/clear_browser_data_overlay.css
index fca5b8d57f8a02f551fd14dafcb920f0ebacf7f2..e15c2b1a3cbe79bb0cc4cc6c8804f8e0c9a23666 100644
--- a/chrome/browser/resources/options/clear_browser_data_overlay.css
+++ b/chrome/browser/resources/options/clear_browser_data_overlay.css
@@ -48,10 +48,41 @@ input[type=checkbox]:not(:checked) ~ .clear-browser-data-counter {
display: none;
}
+#some-stuff-remains-footer {
+ display: block;
+}
+
#some-stuff-remains-footer > p {
- margin: 0;
+ -webkit-padding-start: 33px;
}
#some-stuff-remains-footer button {
padding: 0;
}
+
+#clear-browser-data-history-footer {
+ background-image: url(googleg.svg);
+ background-position: left center;
Dan Beam 2016/03/31 04:47:00 what extra space is there? I.e. why do you need th
msramek 2016/03/31 18:19:56 The footer also contains text (starting at -webkit
+ background-repeat: no-repeat;
Dan Beam 2016/03/31 04:47:00 nit: background shorthand
msramek 2016/03/31 18:19:56 Done.
+ background-size: 16px 16px;
+ margin: 0 0 0.8em 0;
+ min-height: 16px;
msramek 2016/03/21 16:30:25 Since the icons are only in the background, I adde
+}
+
+#clear-browser-data-general-footer {
+ background-image: url(info.svg);
+ background-position: left center;
+ background-repeat: no-repeat;
+ background-size: 18px 18px;
+ margin: 0;
+ min-height: 18px;
+}
+
+html[dir='rtl'] #clear-browser-data-history-footer,
+html[dir='rtl'] #clear-browser-data-general-footer {
+ background-position: right center;
+}
+
+#clear-browser-data-history-notice {
+ width: 300px;
+}

Powered by Google App Engine
This is Rietveld 408576698