Closed (fixed)
Project:
Responsive and off-canvas menu
Version:
4.4.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
30 Sep 2020 at 13:04 UTC
Updated:
16 Jul 2021 at 17:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
tancCan you provide a link to the page on the mmenu site with this feature so I know what you mean?
Unfortunately I don't have time to add new features but if you can debug why you can't get it to work and propose a change to the default template (if that is the issue) then I'd be open to it.
I assume you've tried to modify the config or options as per the README.md?
Comment #3
tancDefinitely not critical as this is a feature request not a bug which completely stops the module from working.
Comment #4
devKhairul commentedHi, sorry for choosing the wrong priority.
I got it to work by adding the option to the config file. It works great now. You can mark this Fixed too. Thanks!
Comment #5
tancGood stuff! Could you share your code so others can benefit?
Comment #6
devKhairul commentedOf course.
I just added the 'slidingSubmenus: false' option to the variable (in responsive_menu.config.js) which essentially opens the submenu items vertically instead of a slide-out window.
Comment #7
devKhairul commentedI do have a silly question though. I know this change will get overwritten when I update the module; in fact there's an update available right now. What's the best way to handle this? How can I copy the file to my theme so it doesn't change my custom configurations? Thanks!
Comment #9
astoker88 commentedtrying to follow this thread - did this get merged in?
If not patch incoming
Comment #10
tancMy comment in #2 is still relevant and I think devKhairul didn't quite understand and modified the config directly in the module by the looks of it.
Here is a link to the relevant section of the README https://git.drupalcode.org/project/responsive_menu/-/blob/4.4.x/README.m...
Comment #11
astoker88 commentedtrying to follow this thread - did this get merged in?
If not patch incoming
Comment #12
tancI'm not going to merge it. Please see the README that I linked to for instructions on how to add your custom config for your needs.
Comment #13
astoker88 commentedComment #14
astoker88 commented@tanc apologies didnt fully read your last comment. works fine for me adding "slidingSubmenus" => FALSE,