Change layout on node/add to support vertical tabs?

domesticat - January 6, 2009 - 14:07
Project:LiveJournal CrossPoster
Version:6.x-1.4
Component:User interface
Category:feature request
Priority:minor
Assigned:valthebald
Status:reviewed & tested by the community
Description

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?

#1

valthebald - February 2, 2009 - 12:25

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

domesticat - February 2, 2009 - 15:24

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

valthebald - February 3, 2009 - 11:37
Assigned to:Anonymous» valthebald
Status:active» needs review

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.

AttachmentSize
ljxp.fieldset.patch 695 bytes

#4

domesticat - February 3, 2009 - 15:21
Status:needs review» reviewed & tested by the community

Patch works beautifully. RTBC. Implement if you like.

 
 

Drupal is a registered trademark of Dries Buytaert.