Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi Guys,
I'm attaching a small patch here to add basic mobile menu support to Zen themes.
It uses the menu title as the trigger and then expands and contracts the menu when clicked.
It's set to kick in at 480px (same as when the menu position changes in Zen).
This is my first time submitting a patch to Zen so am not sure if I was supposed to patch Zen or STARTERKIT, so I went with STARTERKIT since it's a replica of the Zen theme I used to create this.
To use this, make sure you comment in the line for js/script.js in the theme's .info file.
Comment | File | Size | Author |
---|---|---|---|
#1 | zen-basic-mobile-menu-support-2468307-1.patch | 1.86 KB | markconroy |
Comments
Comment #1
markconroy CreditAttribution: markconroy at Annertech commentedComment #2
JohnAlbinSorry I didn't see this earlier.
Issues with patches should be marked "needs review"! :-) Also, it should go in the latest -dev release rather than marked against the latest stable release.
Comment #3
markconroy CreditAttribution: markconroy at Annertech commentedThanks John,
I'll amend and re-submit.
Comment #4
JohnAlbinThe problem with this line is that it will add a click handler on page and every time an AJAX response is returned. You need to add the context variable as the second parameter of $().
Comment #5
markconroy CreditAttribution: markconroy at Annertech commentedThanks John. Been working on a full rewrite of this. Will take that on board as well.
Comment #6
JohnAlbinNo new features are being accepted for Drupal 7 versions.