On #2042807: Convert search plugins to use a ConfigEntity and a PluginBag I did a survey of "What do we call the administrative label of things?". I looked at "things" such as Image styles, Content types, Views, and Display modes, where an admin can create multiple "things" and each one needs an administrative label as well as a machine name.

In most cases, the administrative label is called a "name". So for instance, on admin/structure/views, the column header on the table of views is "Name", and if you are creating a view, the edit field label is "View name". For node content types on admin/structure/types, the table column header is "Name", and if you are editing one, the field is also labeled "Name". For image styles on admin/config/media/image-styles, the table column header is "Style name", and when you are editing one, the field is called "Image style name".

Then in all cases, there is also a "machine name" that gets generated automatically from the "thing name", which means you have a "thing name" and a "machine name", and it seems pretty consistent.

But when you're looking at View modes or Form modes, the table header is Label and the edit field is called Label, while the machine name is still called "machine name". It's just a small cognitive disconnect -- it doesn't match what other parts of Drupal are using, and you have "machine name" vs. "label".

So my suggestion is to change the column headers on
admin/structure/display-modes/form
admin/structure/display-modes/view
to say "Name" and the edit field titles when you are editing to say "View mode name"/"Form mode name". Or on the edit screen, it could just simply say "Name".

CommentFileSizeAuthor
#6 2156657-overview.png63.18 KBswentel
#6 2156657-form.png37.04 KBswentel
#3 2156657-3.patch1.19 KBswentel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

The method is Entity::label(). I don't think that's changing to Entity::name() anytime soon. Same with the actual stored property.

So maybe we should fix it, but the other direction.

jhodgdon’s picture

I disagree with this logic.

Just because it is called Entity::label() to programmers does not mean it should not be called "Name" in the user interface. I think changing things like "View Name" and "Name" for content types to "Label" would be a step backwards in having clear user interface text.

swentel’s picture

Status: Active » Needs review
FileSize
1.19 KB
swentel’s picture

Issue tags: +Drupalaton 2014
jhodgdon’s picture

A screen shot of both of the Manage pages would help. :)

swentel’s picture

FileSize
37.04 KB
63.18 KB

Both overviews use the same code, so one overview screenshot should be fine :)

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 55b18b6 and pushed to 8.0.x. Thanks!

  • alexpott committed 55b18b6 on 8.0.x
    Issue #2156657 by swentel | jhodgdon: Use the word "name" instead of "...

Status: Fixed » Closed (fixed)

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