New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Header navigation hover state overflows the header #221
Comments
@nahurst thanks for pointing this out! I believe this is an intentional design decision by the original creator, hence the drop-shadow effect around the button. @minamarkham @Cara-Jo thoughts on this? Is this something we want to change? |
I also thought this was intentional
|
It is an intentional design decision. H0wever, the drop shadow is not consistent with any other branding guidelines that we have in place. So I'm up for removing the drop shadow, but I kind of like the tab look. |
I like the newer/aligned version. |
Is that new example totally aligned? Seems a pixel off still... |
I think it is. It was a quick and dirty in the browser mockup. |
OH okay! That's fair just hoping that was seen by all haha I feel like I can't unsee the tabs now though! Aligned seems too plain... |
Where do we stand with this? Love/hate the tab look? Let's finalize or close this :) |
Is there a place we can see the tab version w/o the drop shadow? |
^ woops haha put a space between @ and media to not activate the tag |
👍 to removing the drop shadow! thanks @kstack7. I could take or leave the oversized tabs - I agree with @nahurst that they looked a bit like a bug before, but they look cleaner without the drop shadow so I'd be okay leaving them if @Cara-Jo & others would like to, or I'd be down for a slimmed version like what @minamarkham proposed. tl:dr; win-win-win but drop the drop shadow? |
Yeah I think what kept them from looking like a bug to me is the fact the text centered when hovered/active. Otherwise I think it'd look way off. I'm down with dropping the shadow and leaving the rest as it is 👍 |
I feel like it looks a little less polished without the shadow, but agree it doesn't go with anything else. Herm. Either way, another 👍 for tabs :) |
Shadows-be-gone: https://girl-develop-it-staging.herokuapp.com/ |
\o/! On Wednesday, May 13, 2015, Kristen notifications@github.com wrote:
Vanessa Hurst Girl Develop It http://girldevelopit.com/ teaches women to code in over |
The black hover state you see when hovering over header navigation links is bigger than the header. This happens on the latest Chrome and Firefox on OS X.
The text was updated successfully, but these errors were encountered: