Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: skin/desktop-options.css

Issue 29609587: Issue 6031 - Implement Acceptable Ads notification (Closed)
Patch Set: Addressed comments Created Nov. 21, 2017, 2:56 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: skin/desktop-options.css
===================================================================
--- a/skin/desktop-options.css
+++ b/skin/desktop-options.css
@@ -643,6 +643,40 @@
Acceptable ads
*/
+#tracking-warning
+{
+ position: relative;
+ padding: 1.5rem;
+ margin-bottom: 1rem;
+ border: 2px solid #ffd7a3;
+ background-color: #fefbe3;
+}
+
+#acceptable-ads:not(.show-warning) #tracking-warning
+{
+ display: none;
+}
+
+#hide-tracking-warning
+{
+ position: absolute;
+ right: 1rem;
+ top: 1rem;
+}
+
+html[dir="rtl"] #hide-tracking-warning
+{
+ left: 1rem;
+ right: auto;
+}
+
+#tracking-warning .link
+{
+ color: inherit;
+ text-decoration: underline;
+ font-weight: 700;
+}
+
#acceptable-ads ul
{
position: relative;
@@ -656,7 +690,7 @@
padding-right: 2.2rem;
}
-#acceptable-ads button
+#acceptable-ads ul button
{
position: absolute;
margin-top: 0.3rem;
@@ -939,7 +973,7 @@
}
/*
- Used for translatable screen reader only conten.
+ Used for translatable screen reader only content.
e.g.: Use instead of aria-label to avoid complex attribute value translation
*/
.sr-only
@@ -1336,7 +1370,8 @@
}
#dialog-close,
-#hide-notification
+#hide-notification,
+#hide-tracking-warning
{
border: 0rem;
padding: 0rem;
@@ -1346,7 +1381,8 @@
}
#dialog-close::before,
-#hide-notification::after
+#hide-notification::after,
+#hide-tracking-warning::after
{
content: "";
display: block;
@@ -1478,7 +1514,17 @@
background-image: url(icons/delete.svg?secondary#secondary);
}
+#hide-tracking-warning::after
+{
+ background-image: url(icons/delete.svg?tertiary#tertiary);
+}
+
#hide-notification:hover::after
{
background-image: url(icons/delete.svg?secondary-hover#secondary-hover);
}
+
+#hide-tracking-warning:hover::after
+{
+ background-image: url(icons/delete.svg?tertiary-hover#tertiary-hover);
+}

Powered by Google App Engine
This is Rietveld