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.
I just had an instance where it would be really helpful for the 'inline' option of a jump menu to be configurable. Here's a small patch that makes that possible.
Comment | File | Size | Author |
---|---|---|---|
#11 | ctools-1920690-views-inline-jump-menu.patch | 2.93 KB | jnettik |
#8 | views-inline-jump-menu-config_v3.patch | 3.04 KB | jnettik |
#5 | views-inline-jump-menu-config_v2.patch | 3.33 KB | jnettik |
views-inline-jump-menu-config.patch | 1.49 KB | jnettik | |
Comments
Comment #2
jthorson CreditAttribution: jthorson commentedBot issue. Requeued.
Comment #3
jnettikChanging status back.
Comment #4
dawehnerThere are two integrations into jump menus: The normal style and the summary style. It would be cool if you could provide a patch for both of them.
What is the default configuration in ctools? I think we should not change existing sites, so we better set it to FALSE?
Comment #5
jnettikI updated the patch to include both jump menus. ctools default is true so I left that setting as is for now. Let me know if you think it should change and I update that.
Comment #7
dawehnerYour patch should be relative to the views root directory not your root of your drupal installation.
Comment #8
jnettikOops, sorry about that. Here is an updated patch.
Comment #9
dawehnerThis looks pretty good!
Comment #10
dawehnerThanks for writing the patch! Committed and pushed to 7.x-3.x
Let's add this to the ctools part of it in 8.x
Comment #11
jnettikSo I ported this over. I'm having some issues testing it as the latest pull of core gives me a 500 internal error.
Comment #13
jnettik#11: ctools-1920690-views-inline-jump-menu.patch queued for re-testing.
Comment #15
DamienMcKennaThe 8.x-1.x branch has been abandoned, 8.x-2.x is being rewritten from scratch, so I've moved this to the 8.x-2.x branch.