As per a discussion in IRC, it'd be nice if views provided more powerful control over the parent menu items that are generated when a view provides a menu tab, and is selected as the default menu tab at a given location. attached is a proposed screenshot of the views UI for these changes (to get initial feedback on that) while i work on the underlying code to make this possible.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | views_default_tab_parent.patch_5.txt | 1.16 KB | dww |
| #6 | views_default_tab_parent.patch_4.txt | 9.5 KB | dww |
| #5 | views_default_tab_parent.patch_3.txt | 9.16 KB | dww |
| #3 | views_default_tab_parent.patch_2.txt | 8.21 KB | dww |
| #2 | views_default_tab_parent.patch.txt | 8.01 KB | dww |
Comments
Comment #1
merlinofchaos commentedThe mockup looks good.
Comment #2
dwwadded a box to specify the weight of the parent item, if that's a tab. here's the code.
Comment #3
dwwhere's a better version of views_update_15() that would work on pgsql. ;)
Comment #4
dwwwhoops, i didn't realize the export stuff had to be coded manually. so, the new fields aren't being exported now. i'll re-roll.
Comment #5
dwwnow with support for exporting. ;)
Comment #6
dwwbased on feedback from webchick, a few changes to the UI:
it's still a little confusing, i'll admit, but this is certainly better. ;) obviously, there are all sorts of cool JS things we could do to hide/show form elements that aren't relevant depending on higher-level choices, but that's clearly feature creep and beyond the scope of this issue...
Comment #7
merlinofchaos commentedApplied to 5.x branch. Thanks!
Comment #8
dwwsweet, glad this made it in. ;) excellent news.
Comment #9
dwwoh crap, i just realized there's a bug in here. i forgot to fix the export code when i made the changes for #6, so the type for the parent item for the default tab isn't being exported properly. :( this causes all sorts of grief if you export or try to define a default view using this stuff. luckily, it's a trivial fix.
Comment #10
dwwnow fixed by the duplicate http://drupal.org/node/149829. *shrug*
Comment #11
(not verified) commented