Reviewed & tested by the community
Project:
Jump
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
16 Aug 2010 at 15:29 UTC
Updated:
15 Nov 2011 at 17:36 UTC
Jump to comment: Most recent file
Comments
Comment #1
alb commentedComment #2
greenc commentedWe like the module very much and use it ourselves. Always grateful for contributed code. But, I'd like to second the motion above...
As a case in point, we use it for a menu, but it's auto-created 73 different blocks between our menus and taxonomies. We can certainly modify the code, but just wanted to agree with alb that the module would be much improved without all of the extra blocks. Auto-creating blocks in this context is problematic, especially for large sites. The result is that it takes the blocks page an eternity to load. You're by no means alone, some other modules do (or did) this as well, Tagadelic comes to mind.
My guess would be that a little checkbox in the admin/build/menu-customize/menu-admin/edit section asking if one would like to create a jump menu block would be great... or perhaps better still, adding a list of checkboxes to the Jump admin page. I would suggest the best default would be to create no blocks until the user makes the first move.
Just a couple suggestions from the user community...
Thanks again
Comment #3
marcp commentedGood point -- I haven't used Jump lately on a site with loads of menus and taxonomies.
If anyone can supply a patch I'd gladly review and commit it.
Comment #4
marcp commentedMoving to 6.x-2.x.
Comment #5
venusrising commentedIs there any way to remove some of the unused blocks as it its really cumbersome if you only need it for one or two menus.
Thanks
Comment #6
grasmash commentedrolled against 2.x-dev. you can now select which menus and vocabs appear as jump blocks.
Comment #7
grasmash commentedchanging status. please review.
Comment #8
venusrising commented@madmatter23 can we apply this patch to the stable release? as we are using 6x1.1 since it is a production site.
Thanks for the work and the help. Happy to test and let you know.
Cheers
Comment #9
grasmash commentedI think that the patch should work against any 6.x-2.x. It's quite easy to apply it to the 6.x-1.x branch as well, the main difference being that hook_block now uses a 'switch' rather than 'if else.'
Comment #10
grasmash commentedI just rolled the patch against 1.x. Please try it out and let me know if it works -- I don't have 1.x installed on my site.
Comment #11
grasmash commentedyou should also replace jump_uninstall with this in your jump.install file
This will ensure that stray variables aren't left in your database after uninstalling the module.
Comment #12
venusrising commented@madmatter23 I applied patch all works well so far. Please ignore screenshot. I was patching manually and missed something, went back and got it fixed.
Comment #13
venusrising commentedSWEET! Works awesome! Thanks so much what a great enhancement. BTW the module admin mentions the current page setting but how would one add text that says select category or whatever to a dropdown?
Comment #14
grasmash commentedThat feature, along with a few others, has been implemented in the 2.x branch. I'd suggest upgrading rather than manually patching the 1.x branch.
Comment #15
venusrising commented@madhatter23 we will keep stable version that we patched until 2 is stable as it is a production site. Thanks.
Comment #16
shunting commentedThis is a nifty little module that "just works." Thanks. It solved my exact problem.
Comment #17
grasmash commentedchanging status after a number of people in community have reported successful patches. rtbc.
anything holding this back?