Closed gzabrisk closed 11 years ago
I'm open to suggestions there. I looked at reversing the colors (dark text on white background) for the current state, but it seemed way too drastic. Since the contrast in the menu is already quite high, it's difficult to increase it.
I was looking at the mobile menu and I agree that it doesn't work there. I was thinking of just removing that style until the full-width menu appears. You lose a little bit of information at the small sizes, but I think I'd be okay with that.
Here's an idea. This is something I tend to do when creating menus:
First, set the default color of the navigation text to something not so bright but still readable and accessible. That way, the current nav item can be set to white, which makes it stand out. Also give the current nav item a slightly lighter or darker background. The combination of the brighter text and the slightly different background color will allow it to stand out from the other nav items enough without too drastic a change. Here's a example I threw together using the Chrome Inspector Tools:
I'm not in love with the colors I chose above (just threw it together) but it gets the idea across.
I like it. I'll give that a try. Thanks for the contribution. Are you interested in directly editing? I'd be happy to add you to the group.
Sure! I don't have a lot of time to contribute, but I'll be happy to help how I can.
Added the text colors, as well as a slight background gradient. I think it's working pretty well, so I'll close this one for now, but feel free to continue suggesting improvements if you see them.
The new responsive templates are looking great and I've excited to see these coming out. I do wonder though if the simple, lighter color line under the current navigation item is too subtle. It's especially noticeable in the mobile version navigation. The line looks more like a separator between the nav items than an indicator that the above nav item is current.
Just a thought!