Posted by domesticat on January 6, 2009 at 2:07pm
| Project: | LiveJournal CrossPoster |
| Version: | 6.x-1.6 |
| Component: | User interface |
| Category: | feature request |
| Priority: | minor |
| Assigned: | valthebald |
| Status: | closed (fixed) |
| Issue tags: | vertical tabs |
Issue Summary
I'm a recent convert to the Vertical Tabs project, and I really like how it simplifies the user interface. Any chance you'd consider modifying how the LJXP checkbox is presented so that Vertical Tabs will pick it up?
Comments
#1
I don't know vertical tabs internals, but if it uses standard drupal Form API, then it's all just a matter of themeing.
LJXP doesn't contain any HTML to present checkbox, it just adds elements to node editing form, and it's theme's responsibility to render the form.
P.S. Sorry for late response - didn't get notification about new issue on LJXP project
#2
Take a look at the module page for vertical_tabs and you'll see a screenshot. It applies some nice CSS to fieldsets and stacks them cleanly. vertical_tabs can't access the output of LJXP because the checkbox is not contained in a fieldset.
#3
try this patch. It places ljxp form addition into fieldset, if I got it right, this should work with vertical tabs.
Sorry, can't test it by myself.
Please tell me if it worked.
#4
Patch works beautifully. RTBC. Implement if you like.
#5
Note, you should add '#group' => 'additional_settings' to that fieldset as well for it to be tabified by default.
#6
I think not all people would like to do it by default.
Better option is to modify this in theme's template.php if necessary
#7
#8
Automatically closed -- issue fixed for 2 weeks with no activity.