The following is a patch that improves the interface for the Behaviors in the Preset form. It basically puts everything into a table.

A thing to note is that I am attempting to use CTools to hide behavior options when behavior is not active, but this does not seem to be working. Either way, this is a definite improvement to the huge list of Behaviors that is currently there.

Comments

tmcw’s picture

Looks good. I'll see if ctools dependencies will work for this.

zzolo’s picture

Just to let you know. The Ctools dependency stuff is in there, at least from what I understood of the documentation. But fieldsets behave a little differently, apparently.

zzolo’s picture

StatusFileSize
new6.43 KB

Figured it out. You can't actually use a fieldset type, it has to be a hidden type with a prefix (and then another markup type for the suffix). Quite a hack, but still simpler that writing from scratch. I think this is a pretty good improvement.

zzolo’s picture

Issue tags: +beta blocker

tagged as beta blocker

tmcw’s picture

How is this a beta blocker? It's a good but non-essential improvement.

zzolo’s picture

Because its in the queue ready to be reviewed and I would like to see it in. I just don't see the point in putting this off as it is really close to finishing.

tmcw’s picture

Status: Needs review » Reviewed & tested by the community

Okay, tested this and it work as advertised, feel free to commit.

zzolo’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for the review. Committed:
http://drupal.org/cvs?commit=416560

Status: Fixed » Closed (fixed)
Issue tags: -beta blocker

Automatically closed -- issue fixed for 2 weeks with no activity.