Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The output to the account page could do with work on several fronts.
First, what we output ends up inside a DL no matter what we do -- it's just the way user module works. Currently we get bare text inside a DL, which is invalid HTML.
Second, I'm really not sure about the grouping being that high up. Top-level categories on the account page should be for pretty big things: both user and blog modules put their stuff under 'History' for instance. Currently we have a top-level category for each vocabulary if grouping is turned off. It would make more sense to have one heading for 'Taxonomy terms', and then DL headings for each vocabulary. (Though: what that heading should actually be: #689508: change user account form fieldset label).
Which brings us to the grouping. This isn't a feature I use, but the idea seems to be to show all terms in one list, with no vocabulary headings. That's easily accommodated in the change.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 724880.user_terms.user-account-page.patch | 3.83 KB | joachim |
Comments
Comment #1
joachim commentedPatch also changes from a BRs separating terms to using theme_item_list() on them.
Comment #2
joachim commented#724880 by joachim: Fixed use of hook_user view API to produce compliant HTML and grouped all terms under a single top-level category.
Committed the patch.