[UX] Contact form categories: recipients should be rendered as a list

[klonos]

Part of the #3023 META...

Current situation (single line of text w/o spaces):

These should be rendered as a list.

---

PR backdrop/backdrop#2256

[opi]

from #2872 (comment)

We could be sorting email addresses alphabetically for an added bonus. This would help in cases where these lists get long. I would first go by grouping by domain, then alpha sorting for each domain.

I agree with rendering recipients as list, but sorting addresses alphabetically, and/or by domain can be confusing for user : The recipients order would be different between the overview page and the category edit form. Not sure this is good UX.

[opi]

backdrop/backdrop#2256 simple PR with a line break added for each recipient address.

[klonos]

Agreed about the alpha order, but I still think that it'd be better to have the emails rendered as a list. That is the best option semantically (because it is a list of things), and this is the pattern being followed elsewhere in the admin UI:

[opi]

PR updated with a proper <ul><li> structure. Note that this structure is used even with only one recipient.

[klonos]

Note that this structure is used even with only one recipient.

That's what currently happens with the list of roles in the user account page too. I personally do not find this annoying in these cases. It is just a clearer indication that this specific thing can have more than one items.

[klonos]

I have left a minor comment in the PR, but in general RTBC from me 😄

[opi]

Using theme('item_list') now as suggested by @klonos ; I guess we're RTBC now, if you can review a last time and set the proper label, thx !

[klonos]

Poifect! RTBC from me 👍

[quicksketch]

Thanks @opi and @klonos! Merged backdrop/backdrop#2256 into 1.x for 1.11.0.