Recommendation:
"Change order to ask for title field first.

Autofill description based on title."

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

in block, title isn't a required field. should not ask it first.

Bojhan’s picture

Yes, 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.

droplet’s picture

but Desp is the TITLE when it on backend block admin page.

mrfelton’s picture

Status: Active » Needs review
FileSize
1.31 KB

I 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.

attiks’s picture

Status: Needs review » Reviewed & tested by the community

Patch looks solid, motivation in #2 is spot on

mrfelton’s picture

Ok, just figured out git format-patch - here a nice version! (exact same code changes)

yoroy’s picture

The 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:

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks.

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