Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
block.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
21 May 2011 at 20:42 UTC
Updated:
29 Jul 2014 at 19:35 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
droplet commentedin block, title isn't a required field. should not ask it first.
Comment #2
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 commentedbut Desp is the TITLE when it on backend block admin page.
Comment #4
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 commentedPatch looks solid, motivation in #2 is spot on
Comment #6
mrfelton commentedOk, just figured out git format-patch - here a nice version! (exact same code changes)
Comment #7
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 commentedCommitted to 7.x and 8.x. Thanks.