The attached patch allows excluding menu_minipanels from specified paths.
It uses the same approach as block visibility settings.
It adds a new textarea to the admin settings form only, to allow specifying pages to exclude ( see screenshot )
We use this to prevent the additional markup and js etc.. being added on pages where it's not required.
Please review and commit when possible.
Thanks!
DT
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | menu_minipanels-n1440474-7-d6.patch | 3.81 KB | damienmckenna |
| #6 | menu_minipanels-n1440474-6-d7.patch | 3.88 KB | damienmckenna |
| #5 | 7.x-1.x-1440474-exclude_pages.patch | 2.77 KB | josh waihi |
| #4 | 7.x-1.x-1440474-exclude_pages.patch | 2.77 KB | josh waihi |
| #1 | menu_minipanels_exclude_pages_setting-1440474-1.patch | 2.56 KB | jweowu |
Comments
Comment #1
jweowu commentedRe-rolled patch to apply against the project repository.
Comment #2
davidwhthomas commentedThanks, btw loving this patch, hopefully can be committed upstream soon.
Comment #3
damienmckennaThis should replace the "hide on admin pages" option and default to "admin\nadmin/*". Thanks :)
Comment #4
josh waihi commentedRerolled with rc5 and replaces menu_minipanels_show_admin.
Comment #5
josh waihi commentedGot the admin check condition around the wrong way. Fix attached.
Comment #6
damienmckennaI've adjusted it slightly - the variable is now called 'menu_minipanels_exclude_paths', I optimized the menu_minipanels_excluded_path() function slightly and I added an update script to replace the old variable.
Comment #7
damienmckennaD6 patch.
Comment #8
damienmckenna@davidwhthomas: Thanks for putting the work into this feature, I've committed it to both the D6 and D7 versions.
Comment #9
davidwhthomas commentedHey, glad to hear it!
Thanks for your work also on this cool module.
cheers!
DT