[Theme: Twenty Fourteen] Changing border radius on sub menu from main nav.

Hey there.

I tried to take a look http://booteeki.com/test/ but you have a fatal error, you’re trying to redeclare a function (mp_list_products) twice, once from here:

/marketpress/marketpress-includes/template-functions.php

And then in here:

/Booteeki Theme/functions.php

Let me know when it’s fixed up.

Cheers.

Hey again.

Probably should remove the space out of the filename there. Use a hyphen or remove the space.

So this is the CSS you refer too?

.site-navigation .current_page_item > a,
.site-navigation .current_page_ancestor > a,
.site-navigation .current-menu-item > a,
.site-navigation .current-menu-ancestor > a {
	color: black;
	background-color:#C89FC1;
	border-bottom-left-radius:85%;
	border-bottom-right-radius:85%;
	height:8px;
}

I see originally you mentioned about targeting the current item. I just looked on your site and it seems all the child items in your menu also have the same class, this is why it applies to them all and not just one.

I tested in the default Twenty Fourteen theme that your child is based off and that behaviour doesn’t occur for child menu items unless they are actually the current page. Are you doing something custom with menus there?

Cheers.

Hi there joekober,

I’m Jose from WPMU DEV.
Tim asked me to look into this and get you a fix.

I found a bug in how the wp_nav_menu function is being hooked. We will push a fix as soon as possible.

In the meantime, you can apply the fix in your installation by making the following changes into the file ..\wp-content\plugins\wordpress-ecommerce\marketpress.php:

– In line 160 replace this:
add_filter( 'wp_nav_menu_objects', array(&$this, 'filter_nav_menu'), 10, 2 );
with this:
add_filter( 'wp_get_nav_menu_items', array(&$this, 'filter_nav_menu'), 10, 2 );

– In line 1487 replace the entire function filter_nav_menu with this fixed version:

function filter_nav_menu($list, $menu, $args = array()) {
		$store_object = false;

         if ($args->depth == 1)
            return $list;

            //find store page
            $store_url = mp_store_link(false, true);
            $store_page = get_option('mp_store_page');
            foreach($list as $menu_item) {
                if ((isset($menu_item->object_id) and $menu_item->object_id == $store_page) || $menu_item->url == $store_url) {
                    $store_object = $menu_item;
                    break;
                }
            }

            if ($store_object) {
                $obj_products = clone $store_object;
                $obj_products->title = __('Products', 'mp');
                $obj_products->menu_item_parent = $store_object->ID;
                $obj_products->ID = 'products-subm';
                $obj_products->db_id = 'products-subm';
                $obj_products->post_name = 'products-subm';
                $obj_products->url = mp_products_link(false, true);
                $obj_products->current = (get_query_var('pagename') == 'product_list') ? true : false;
                $obj_products->current_item_ancestor = (get_query_var('pagename') == 'product_list') ? true : false;
                $obj_products->menu_order = 997;
                $obj_products->object_id = '99999999997';
                $obj_products->object = 'custom';
                $list[] = $obj_products;

                 //if cart disabled return only the products menu item
                if ($this->get_setting('disable_cart'))
                     return $list;

                $obj_cart = clone $store_object;
                $obj_cart->title = __('Shopping Cart', 'mp');
                $obj_cart->menu_item_parent = $store_object->ID;
                $obj_cart->ID = 'shopping-cart-subm';
                $obj_cart->db_id = 'shopping-cart-subm';
                $obj_cart->post_name = 'shopping-cart-subm';
                $obj_cart->url = mp_cart_link(false, true);
                $obj_cart->current = (get_query_var('pagename') == 'cart') ? true : false;
                $obj_cart->current_item_ancestor = (get_query_var('pagename') == 'cart') ? true : false;
                $obj_cart->menu_order = 998;
                $obj_cart->object_id = '99999999998';
                $obj_cart->object = 'custom';
                $list[] = $obj_cart;

                $obj_order = clone $store_object;
                $obj_order->title = __('Order Status', 'mp');
                $obj_order->menu_item_parent = $store_object->ID;
                $obj_order->ID = 'order-status-subm';
                $obj_order->db_id = 'order-status-subm';
                $obj_order->post_name = 'order-status-subm';
                $obj_order->url = mp_orderstatus_link(false, true);
                $obj_order->current = (get_query_var('pagename') == 'orderstatus') ? true : false;
                $obj_order->current_item_ancestor = (get_query_var('pagename') == 'orderstatus') ? true : false;
                $obj_order->menu_order = 999;
                $obj_order->object_id = '99999999999';
                $obj_order->object = 'custom';
                $list[] = $obj_order;
            }

            return $list;
	}

Let us know if it works for you. We’ll keep you posted on the release of the fixed version.

Cheers!

Glad to hear that it works now!

The arrow is showing up because now the class .menu-item-has-children is being added to the element, as expected.

You should override the style for the following selectors:

.menu-item-has-children > a
.menu-item-has-children > a:after

For instance, you can use the following (need testing):

.menu-item-has-children > a {
    padding-left: 12px !important;
}
.menu-item-has-children > a:after {
    display: none !important;
}

There might be something missing. Please try and let me know.

Cheers!

Hey joekober.

Thanks for your patience on this one, glad we could get it sorted for you. The next release should contain this fix. 🙂

Take care.