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

Unified Diff: skin/popup.css

Issue 29321084: Issue 2195 - Added notification opt-out (Platform) (Closed)
Patch Set: Created June 25, 2015, 1:09 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
« options.js ('K') | « popup.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skin/popup.css
===================================================================
--- a/skin/popup.css
+++ b/skin/popup.css
@@ -63,7 +63,6 @@
#notification
{
- display: none;
position: relative;
margin-top: -70px;
min-height: 50px;
@@ -96,7 +95,7 @@
color: #fff;
}
-#close-notification
+#notification-close
{
cursor: pointer;
display: inline-block;
@@ -109,6 +108,40 @@
background-position: -34px -89px;
}
+#notification.closing
+{
+ border-color: #000;
+ background-color: #020D14;
+}
+
+#notification.closing #notification-content,
+#notification:not(.closing) #notification-close-content
Sebastian Noack 2015/06/25 13:55:04 Isn't the |:not(.closing)| redundant here?
Thomas Greiner 2015/06/25 16:48:45 No, because if #notification has the class "closin
+{
+ display: none;
+}
+
+#notification-close-content li
+{
+ padding: 7px 7px 7px 17px;
+ color: #FFF;
+ text-decoration: underline;
+ white-space: inherit;
+ border: none;
+}
+
+#notification-close-content li::before
+{
+ content: "\0203A";
+ display: inline-block; /* prevent pseudo element from being underlined */
+ width: 10px;
+ margin-left: -10px;
+}
+
+#notification-close-content li:hover
+{
+ background-color: rgba(255, 255, 255, 0.3);
+}
+
footer
{
cursor: pointer;
« options.js ('K') | « popup.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld