Hi, thanks for the great implementation. Easy to use and plugs straight into drupal.

One annoyance I have though is that it's a bit redundant to copy over the css if you don't like the default yui stuff.

I took out line 70 in the module to get rid of Sam's Skin for the moment, but a better way might be to add an option to use, or not use, Sam's Skin css.

line 70:
yui_add_css('menu', $yui_source, '/build/menu/assets/skins/sam/menu.css');

Comments

Jonathan.D’s picture

Jonathan.D commented

I thought I'd reiterate. If you take off skins/sam/menu.css the whole menu will be broken. If you don't use the Sam skin you need to use menu-core.css. You can then theme the menu from there.

bakyildiz’s picture

bakyildiz commented

Jonathan, i could not understand what you mean by "If you don't use the Sam skin you need to use menu-core.css"

Can you give more detail. Is there anything that i can correct in code?

asak’s picture

asak commented

subscribing.

bakyildiz’s picture

bakyildiz commented
Assigned: Unassigned » bakyildiz

Hi Jonathan
The yui module has a "Skin to use for YUI" variable. If you enter something different for "yui-skin-sam" the sam skin we be out of use and as you mentioned the menu system will not work any more. But you also mention that using menu-core.css will repair the problem. I tried the menu-core.css bu i could not see that the menu is working. If you can help me to figure out how to have a working menu without sam skin we can add this funtionality to yuimenu.

asak’s picture

asak commented

What's strange is that i could not find any other skins online...