Merge pull request #1645 from tpwwu/issue-1626

gondzo · web-flow · commit 1a54cb108796 · 2018-01-30T19:37:54.000+01:00
Issue #1626 Notifications bell stays active (white) even when there are no new notifications
diff --git a/src/components/NotificationsDropdown/NotificationsDropdown.scss b/src/components/NotificationsDropdown/NotificationsDropdown.scss
@@ -51,6 +51,12 @@
     .icon-ui-bell path {
       fill: $tc-white;
     }
+
+    &:not(.has-unread) {
+      .icon-ui-bell path {
+        fill: $tc-gray-50;
+      }
+    }
   }
 
   .notifications-dropdown-bell.has-unread {
@@ -67,12 +73,6 @@
     }
   }
 
-  .dropdown-wrap.hide .notifications-dropdown-bell:not(.has-unread) {
-    > svg > g > g {
-      fill: $tc-gray-50;
-    }
-  }
-
   .notifications-dropdown-body {
     // window height
     // - 54px dropdown top position

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,12 @@`
`51`	`51`	`.icon-ui-bell path {`
`52`	`52`	`fill: $tc-white;`
`53`	`53`	`}`
	`54`	`+`
	`55`	`+ &:not(.has-unread) {`
	`56`	`+ .icon-ui-bell path {`
	`57`	`+ fill: $tc-gray-50;`
	`58`	`+ }`
	`59`	`+ }`
`54`	`60`	`}`
`55`	`61`
`56`	`62`	`.notifications-dropdown-bell.has-unread {`
`@@ -67,12 +73,6 @@`
`67`	`73`	`}`
`68`	`74`	`}`
`69`	`75`
`70`		`- .dropdown-wrap.hide .notifications-dropdown-bell:not(.has-unread) {`
`71`		`- > svg > g > g {`
`72`		`- fill: $tc-gray-50;`
`73`		`- }`
`74`		`- }`
`75`		`-`
`76`	`76`	`.notifications-dropdown-body {`
`77`	`77`	`// window height`
`78`	`78`	`// - 54px dropdown top position`