Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Recommendation:
"Change order to ask for title field first.
Autofill description based on title."
Comment | File | Size | Author |
---|---|---|---|
#6 | 1164808.6-reorder-block-title-and-description.patch | 1.67 KB | mrfelton |
#4 | 1164808.4-reorder-block-title-and-description.patch | 1.31 KB | mrfelton |
Comments
Comment #1
droplet CreditAttribution: droplet commentedin block, title isn't a required field. should not ask it first.
Comment #2
Bojhan CreditAttribution: Bojhan commentedYes, we should ask it first - even though it is not required. It is likely that people who need the functionality of "no title" are further down the learning curve, thus will explore that it's not required.
There is no convention that, title fields when not required should not come first. It's still the first thing most users think of.
Additionally I feel display settings, like showing or not showing a title should be handled differently - than not filling in one. We should have a consistent way for doing this.
Comment #3
droplet CreditAttribution: droplet commentedbut Desp is the TITLE when it on backend block admin page.
Comment #4
mrfelton CreditAttribution: mrfelton commentedI agree that the title should come first. It makes logical sense, and is consistant with other areas of the admin interface. Just because it's not required, should mean that it moves in the interface. There is no red star to denote the fact that it's not required. Patch attached.
Comment #5
attiks CreditAttribution: attiks commentedPatch looks solid, motivation in #2 is spot on
Comment #6
mrfelton CreditAttribution: mrfelton commentedOk, just figured out git format-patch - here a nice version! (exact same code changes)
Comment #7
yoroy CreditAttribution: yoroy commentedThe original report is a bit too basis, it was unclear to me which screen this is about. But yes, we should switch these fields around:
Comment #8
Dries CreditAttribution: Dries commentedCommitted to 7.x and 8.x. Thanks.