Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.