Skip to content
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

Fix CSS attributes for un-hovered navigation menu items using color_5 #94

Merged
merged 1 commit into from
Oct 21, 2017

Conversation

mpkut
Copy link
Contributor

@mpkut mpkut commented Oct 13, 2017

Our local branding guidelines require a color set that results in fairly dark colors for the top menu items, with a contrasting lighter color for the current item.

We found that when color_5 is set to something other than the default, the current menu item is still rendered as light blue instead of color_5.

stock-color5-not-hovered

Drawing from the code using color_6 immediately above it, this patch to css.tt2 uses color_5 as the background color for the active menu item when it is not being hovered over.

fix-color5-not-hovered

With this update, the colors are as expected and the GUI seems to work correctly when driving under 6.2.22. That is, the current menu item is color_5 (e.g. burnt orange), the rest are color_6 (dark brown), and hovering the mouse over another menu item properly changes the background to color_5 (burnt orange) as well.

--mic--

@ikedas ikedas added the bug label Oct 14, 2017
@ikedas ikedas added this to the 6.2.24 milestone Oct 14, 2017
@ikedas ikedas merged commit ca0c741 into sympa-community:sympa-6.2 Oct 21, 2017
@mpkut mpkut deleted the css_color_5_hover branch October 23, 2017 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants