Problem/Motivation
If you go to the Field List page (admin/reports/fields) will be nice if you can see the entity type where this field is used. Sometimes you can have the same field name in different entity types, so this information can help to fin more quickly the information that you need.
Proposed resolution
Add a new column named Entity type in the Field list page (admin/reports/fields).
Remaining tasks
User interface changes
A new column (Entity type) will be added in the Field list page.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#15 | Field_list___drupal_8_5_x.png | 75.69 KB | xjm |
#12 | drupal-add_entity_type_column-2907110-12-D8.patch | 1.45 KB | adriancid |
Comments
Comment #2
adriancidComment #4
adriancidComment #5
munish.kumar CreditAttribution: munish.kumar commentedI have reviewed your code and it Looks good to me.
+1 RTBC
Comment #6
tim.plunkettThanks! The code looks great, I think this deserves a UX review.
Comment #7
adriancidComment #8
Bojhan CreditAttribution: Bojhan as a volunteer commentedLooks good to me. This is very much a developer facing page, I would say the label "entity" here is justified.
Comment #9
adriancid@Bojhan What's the next step? We need to change this to RTBC?
Comment #10
tim.plunkettThe code is fine, no sense in writing automated tests, and it was given UX sign off.
RTBC from me!
Comment #11
xjmNice improvement!
One suggestion I'd make is making this either the second column, or the second-to-last column. "Used in" specifies bundles, and which bundle it is depends on the entity type, so it seems strange to have the bundle names before the entity type. Since fields can't be shared across entity types, it would make sense for it to be the second column. I think the second column would be best since that would best address the problem statement in the issue summary. If the field name isn't sufficient information, and field name + bundle makes it clearer, it'd be better to have them right next to each other rather than reading all the way to the end of the row.
Four columns across is also a lot on a mobile display. Should we set priorities on some of the columns so that the table is responsive?
Comment #12
adriancid@xjm I put it as second column, you're right is better in this place.
The FIELD TYPE column (that is largest) is defined as RESPONSIVE_PRIORITY_MEDIUM, so when you're in mobile devices sometimes is hidden.
Comment #13
tim.plunkettGood change, it fits much better there.
Comment #15
xjmThanks @adriancid.
Just confirmed this -- here's what it looks like on mobile:
That seems an okay mobile display to me.
Committed and pushed to 8.5.x. Thanks! Also congratulations @adriancid on your second core issue credit. :)
Comment #16
xjmComment #17
adriancidThanks @xjm and @tim.plunkett :-)
I just see now a typo in the issue title :-O
Add a Entity type column in Field list page
But the commit is done.
Sorry for my english ;-)