Issue 6031 - Implement Acceptable Ads notification

skin/desktop-options.css

Index: skin/desktop-options.css
===================================================================
--- a/skin/desktop-options.css
+++ b/skin/desktop-options.css
@@ -620,6 +620,35 @@
   Acceptable ads
  */
 
+#acceptable-ads.show-notification #tracking-notification

[Thomas Greiner, 2017/11/16 19:25:06]

Suggestion: You could avoid setting "display" twice and you wouldn't need to
care about its value if you use `:not()`.

i.e.
#acceptable-ads:not(.show-notification) #tracking-notification
{
  display: none;
}

[saroyanm, 2017/11/17 16:10:39]

Right, done.

+{
+  display: block;
+}
+
+#tracking-notification
+{
+  position: relative;
+  display: none;
+  padding: 1.5rem;
+  margin-bottom: 1rem;
+  border: 2px solid #ffd7a3;
+  background-color: #fefbe3;

[Thomas Greiner, 2017/11/16 19:25:05]

Detail: These colors aren't mentioned in the style guide so we should add them
to the spec.

[saroyanm, 2017/11/17 16:10:39]

Yes in general style guide for this notification is missing from the pull
request. ->
https://bitbucket.org/adblockplus/spec/pull-requests/81/issue-87-new-proposal...

[Thomas Greiner, 2017/11/20 18:36:04]

Acknowledged.

+}
+
+#hide-tracking-notification
+{
+  position: absolute;
+  right: 1rem;

[Thomas Greiner, 2017/11/16 19:25:06]

What about right-to-left environments?

[saroyanm, 2017/11/17 16:10:39]

Done.

+  top: 1rem;
+}
+
+#tracking-notification .link

[Thomas Greiner, 2017/11/16 19:25:06]

Detail: Links also use `cursor: pointer`.

Alternatively, we could use a `<a>` element instead of a `<button>` here.

[saroyanm, 2017/11/17 16:10:39]

`cursor: pointer` is inherited from the class `button`. 
I don't think that we should use `<a>` here, as semantically this is a button
which looks like a link. Similar to the `About Adblock Plus` button in the
sidebar, while for example `Contribute` which looks like a button, is actually a
link,  I tried to be here consistent to that logic.

[Thomas Greiner, 2017/11/20 18:36:04]

Indeed, I missed that.

+{
+  color: inherit;
+  text-decoration: underline;
+  font-weight: 700;
+}
+
 #acceptable-ads ul
 {
   position: relative;
@@ -633,7 +662,7 @@
   padding-right: 2rem;
 }
 
-#acceptable-ads button
+#acceptable-ads ul button
 {
   position: absolute;
   margin-top: 0.3rem;
@@ -1297,7 +1326,8 @@
 }
 
 #dialog-close,
-#hide-notification
+#hide-notification,
+#hide-tracking-notification
 {
   border: 0rem;
   padding: 0rem;
@@ -1307,7 +1337,8 @@
 }
 
 #dialog-close::before,
-#hide-notification::after
+#hide-notification::after,
+#hide-tracking-notification::after
 {
   content: "";
   display: block;
@@ -1444,7 +1475,17 @@
   background-image: url(icons/delete.svg#secondary);
 }
 
+#hide-tracking-notification::after
+{
+  background-image: url(icons/delete.svg#tertiary);
+}
+
 #hide-notification:hover::after
 {
   background-image: url(icons/delete.svg#secondary-hover);
 }
+
+#hide-tracking-notification:hover::after
+{
+  background-image: url(icons/delete.svg#tertiary-hover);
+}