-
Notifications
You must be signed in to change notification settings - Fork 16
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
Make it possible to set ulClass #3
Comments
Hey- After my message back to you earlier, do you still think this is necessary? you should add the class to the parent li that houses the children you want to target. If you doesn't work for some reason, please show me an example. Thanks |
i know is not possible but this is a request to make it possible, and above is an example what i mean, not more. But forgot it i try another solutions. thanks |
I certainly don't think it would be difficult to implement, but in general, if the library has less features then it's easier to maintain and document. I'm all for adding this, but only if it were absolutely necessary to accomplish something. thanks |
Ok. for example the diem Menu make exactly that, but for my actual project i don't want use diem. Your plugin does all the stuff i need but the only one with ul Classes, and your Plugin is absolute perfect. :-) thanks too... |
I think it was very nice if we can set the ulClass of each item.
like this:
$menu->addChild('Homepage', '@homepage');
$menu['Homepage']->ulClass('subnav')->addChild('About Us', '@about_us');
$menu['Homepage']->addChild('other_link', '#');
$menu->addChild('Blog', '@blog');
$menu['Homepage']->ulClass('blog_subnav')->addChild('Archive', '@blog_archive');
$menu['Homepage']->addChild('other_link', '#');
The text was updated successfully, but these errors were encountered: