I love the feature where you can display fields from your bio node on the user registration form. However, I'd like to put all of them in a fieldset. Core already uses a fieldset for the username/email fields, and other modules that optionally inject stuff into this form (for example, OG) use a fieldset as well. If bio used a fieldset, too, it'd look more visually consistent. I can obviously do this in a site-specific module using hook_form_alter(), but it seems like it'd be a nice touch to just do it natively.

Is there interest? If so, I'll cook a patch. Otherwise, I'll just do it locally.

CommentFileSizeAuthor
#1 235970_bio_registration_form_fieldset.patch3.36 KBdww

Comments

dww’s picture

Status: Active » Needs review
StatusFileSize
new3.36 KB

This ended up being ever-so-slightly more complicated than I hoped. user.module removes its own fieldset unless you use hook_user($op = 'register') to embed stuff on the registration form. So, this refactors things a bit to use hook_user() as intended, instead of bolting this onto the form via hook_form_alter(). Note, you also need to use hook_user('insert'), instead of a #submit handler in your form or you get weirdness. Given that, this all seems to work fine in my local testing.

Any objections?

dww’s picture

Status: Needs review » Fixed

Hearing no objections, committed to DRUPAL-5.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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