Closed (fixed)
Project:
Bio
Version:
5.x-1.2
Component:
User interface
Priority:
Minor
Category:
Feature request
Assigned:
Reporter:
Created:
18 Mar 2008 at 20:39 UTC
Updated:
7 Apr 2008 at 19:23 UTC
Jump to comment: Most recent file
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.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 235970_bio_registration_form_fieldset.patch | 3.36 KB | dww |
Comments
Comment #1
dwwThis 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?
Comment #2
dwwHearing no objections, committed to DRUPAL-5.
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.