From e8b00458d33d3947e6e367927470e6f7b2bb1416 Mon Sep 17 00:00:00 2001
From: Srihari Sriraman <srihari@c42.in>
Date: Sat, 24 Aug 2013 22:10:16 +0530
Subject: [PATCH] Adding hover icoons for notifications & conversations

Added white versions of the icons on hover to match the hover of other
header items.
Changing the notifications hover icon to the regular notifications icon when active so
it is visible with the white background.

Fixing the hard coded background-position value for notifications_grey

Can not depend on generated sprite background position as it will change
with changes in icons

Moving the $icons variable to a broader scope

Since the variable is used in the header, the sprites now are imported
before the header

Using the default sprite map instead of the explicitly created sprite map
---
 app/assets/images/icons/mail_grey_hover.png       | Bin 0 -> 265 bytes
 .../images/icons/notifications_grey_hover.png     | Bin 0 -> 449 bytes
 app/assets/stylesheets/application.css.sass       |   2 +-
 app/assets/stylesheets/header.css.scss            |   5 +++++
 4 files changed, 6 insertions(+), 1 deletion(-)
 create mode 100644 app/assets/images/icons/mail_grey_hover.png
 create mode 100644 app/assets/images/icons/notifications_grey_hover.png

diff --git a/app/assets/images/icons/mail_grey_hover.png b/app/assets/images/icons/mail_grey_hover.png
new file mode 100644
index 0000000000000000000000000000000000000000..a4dbff8389e80f9ddc7f7be38b45fadbaff4ba38
GIT binary patch
literal 265
zcmeAS@N?(olHy`uVBq!ia0vp^f<Vm8!3HE}Dx|RkDYhhUcNd2LAh=-f^2tCE&H|6f
zVxW%eAk65bF}ngN$X?><>&kwYLqb4=v*&MYB2Z|Wr;B5V#`(4VhI~y9BCN-4n>weo
zOmbU%eZ!^d3dyLq-^?N+9~xR5x}n43DZuOZ+1&np{<DSmVvdP!4LG0m`*Gpwc~_Je
zcsDS9b2wXmMRPtQZ}a@cyTVpWHfudi+MXa%vFdhbRqn0#LF?3eOuQfPJvopVu>SJ-
z%e!lK|5_k>)o1S#$>mLFzSOTcZlWH!a?^wZ@*D?(gyY|(O?5PLHQotyA%mx@pUXO@
GgeCy38fDo4

literal 0
HcmV?d00001

diff --git a/app/assets/images/icons/notifications_grey_hover.png b/app/assets/images/icons/notifications_grey_hover.png
new file mode 100644
index 0000000000000000000000000000000000000000..4558a55a952727b50099902d9492a5c198bcc62a
GIT binary patch
literal 449
zcmV;y0Y3hTP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV00006VoOIv0RI60
z0RN!9r;`8x010qNS#tmY4c7nw4c7reD4Tcy000McNliru-3S;E5hE%wVDbO}0ZvIo
zK~y-)mD9az(_s+6@gt3f7%QD34iT!-g5cs(EP_g>B05ya(y0{u6D)LzlMp8%kVVi%
zq;{!;lY`D_#X)dVp*4cGQbFP%ihMfww8S65B)8o?ci+3`ULrDrYj}u}7*=o{yV$_v
zp&*#Y4$h;9=Xi#~Ai)K^N3+XT!V9dV_v8H_r?G`wUA80mfo*@K(GPMeL6(w!5kLJa
z<$jQHY@&tlnc#0I4$_me_?-#<g$~wmXx}%7X8P*_E+zYkh{$&_I}tGx5#@*&jfj;4
z2tH#1cW^h^j^bU){0Hlq#=gWV@`0zgi79+SDLEd)6`VsG%b5|k@F-Bl8|3gHEznq|
zCnxa%wWME88+$cU;49q5Vcf?T+{1Ji<P1Kdfv>1$Gqh1lzzeA3My8Sq-eR`b=ql-m
rUSkoHy=MO3+eq2x@FZ1K@3%91f!*Bpp4UEP00000NkvXXu0mjf@RY-)

literal 0
HcmV?d00001

diff --git a/app/assets/stylesheets/application.css.sass b/app/assets/stylesheets/application.css.sass
index 234b7c2f1..a98638d40 100644
--- a/app/assets/stylesheets/application.css.sass
+++ b/app/assets/stylesheets/application.css.sass
@@ -5,9 +5,9 @@
 @import 'new_styles/_spinner'
 
 @import 'sidebar'
+@import 'sprites'
 @import 'header'
 @import 'footer'
-@import 'sprites'
 @import 'opengraph'
 
 @import 'profile.css.scss'
diff --git a/app/assets/stylesheets/header.css.scss b/app/assets/stylesheets/header.css.scss
index cf58dc3fc..b00112e0a 100644
--- a/app/assets/stylesheets/header.css.scss
+++ b/app/assets/stylesheets/header.css.scss
@@ -75,6 +75,11 @@ body > header {
       &:hover .badge_count { background-color: lighten(#A40802, 5%); }
       
       .icons-notifications_grey { height: 16px; }
+
+      &.active .icons-notifications_grey:hover {
+        background-position: sprite-position($icons-sprites, notifications_grey);
+      }
+
       .icons-mail_grey { height: 11px }
 
       a.badge_link {