• Resolved jonny-s

    (@jonny-s)


    Hi,

    I really like your theme. I have three suggestions for the primary-menu.

    First one: I added the following code to have hover-effects for normal user (should not disturb screenreader-behavior, as it only use :hover, not focus, am I right?):

    .main-navigation .menu-item-has-children:hover ul {
    	display: block !important;
    }
    .main-navigation .menu-item-has-children:hover button:hover {
    	box-shadow: none;
    }

    Maybe you want to adopt it.

    The second thing: instead of a border-bottom to a-elements, I suggest to add border to it’s parent li-element. This way, the corresponding button of sub-menus is underlined as well.

    Third, there is a selector not that exactly as needed. You have loaded current page with sub-menu items. These sub-menu items are all underlined the :hover-way. Please alter the selector

    .current-menu-item a, 
    .current_page_item a

    to

    .current-menu-item > span > a, 
    .current_page_item > span > a

    so that current-style only applies to direct child a, not that ones in it’s sub-menu.

Viewing 1 replies (of 1 total)
Viewing 1 replies (of 1 total)
  • The topic ‘Some enhancement-suggestions for primary-menu’ is closed to new replies.