Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This issue is spun off from #2831936-129: Add "File" MediaSource plugin, specifically this point of review:
The media creation form is a hodge-podge of information. The revision information is above / more important than the file itself. I also did not expect the path alias to be this prominent on the form. Do we have an issue to review the media creation UI?
We do now. Let's get UX review on the media creation form, and make as many fixes as possible.
Comment | File | Size | Author |
---|---|---|---|
#14 | review_and_improve_the-2882801-14.patch | 1.4 KB | chr.fritsch |
#9 | media-add-file.png | 277.15 KB | seanB |
Comments
Comment #2
phenaproximaComment #3
phenaproximaComment #4
BerdirI think we have an issue somewhere to make the sidebar that node has generic and re-use in different forms. That would probably help with that?
Comment #5
Wim Leers#4++
Comment #6
Wim LeersPinging @yoroy for a usability review. @yoroy, to review:
/media/add/file
Comment #7
Bojhan CreditAttribution: Bojhan commented@Wim Could we also provide some screenshots? To make sure we capture the "before" state, also easier to review :)
Comment #8
yoroy CreditAttribution: yoroy at Roy Scholten commentedApologies, I missed that this was assigned to me. Will review in the coming days. Screenshoulds certainly help.
Comment #9
seanBI think it would already help if we have a sidebar like the node form has. We could optionally remove the name field since we can auto generate that (but that is probably another issue).
Comment #10
marcoscano#2882473: Hide the media name basefield from the entity form by default is very likely related (or maybe even duplicate)
Comment #11
yoroy CreditAttribution: yoroy at Roy Scholten commentedShow required fields first. Move "file" below "name". Or potentially even above it. It's the most important one.
So is "Name" not the same as "Title" on other entities? It's similar to Title in that it is not available on the "manage fields" page. Why "name" then and not "title"? Not sure about auto hiding it, I imagine there's enough cases where I'd want to be able to look up an asset by it's title.
I wish we did something about that three lines of description text below a file/image upload field. Especially with this wide selection of allowed file formats it would make more sense to not show it at all initially. Then use an inline form error message to list these if a not-allowed file format was provided. Something for the redesign issues for those fields I guess.
Revision log message should be part of the vertical tabs.
Oh dear, I designed those file icons ages ago, we really should update those. Too small, too ugly, not useful.
About the sidebar like on the content creation form, that could work here as well. I think the layout people were thinking that whole sidebar whould have to go again when actual layouts can be applied to these pages.
I'd say:
Must:
Should
Comment #12
Wim LeersThank you so much, @yoroy!
Comment #13
Wim LeersFor the sidebar, this probably needs #2803875: Node form meta information should not come from a theme.
Comment #14
chr.fritschHere is a first patch.
1. I moved the revision items into the vertical tabs
2. This automatically sorted the file field below the name field
3. I made the "name" configurable on the "Manage display" page, but it's in the hidden region per default
IMHO "name" is in case of media more intuitive as "title". Media name just sounds more correct for me, than media title. But technically i guess it's the same.
Comment #16
BerdirManage display isn't related to the media *form*? The widget is already configurable.
And hiding it there already has a separate issue: #2882473: Hide the media name basefield from the entity form by default, with an alternative suggestion based on something we implemented in a project from @slashrsm.
Comment #18
vijaycs85Right, the revision log message box appearing on top was a bug and fixed in #2912333: Revision log field doesn't appear under 'Revision Information' tab everything else is configurable at form display as per #16. Closing this issue for now.