Media Editor Modal: Reorder details fields so the editable regular layout fields appear at the top#78792
Merged
andrewserong merged 4 commits intoJun 3, 2026
Conversation
andrewserong
added
[Type] Enhancement
6 tasks
|
Size Change: -2 B (0%) Total Size: 8.21 MB 📦 View Changed
ℹ️ View Unchanged
|
andrewserong
force-pushed
the
update/media-editor-modal-reorder-details-fields
branch
from
02012d1 to
ffd444d
Compare
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Contributor
Author
I've tried simplifying things a bit so this PR now removes a bunch of overrides and relies on
I think this is compelling, and I like how much is simplified. I'll give it some more testing and see if we can also improve the filename field a bit more, too, through the week. |
|
Flaky tests detected in ffd444d. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/26740391799
|
andrewserong
force-pushed
the
update/media-editor-modal-reorder-details-fields
branch
from
ffd444d to
a8ee5e6
Compare
Contributor
Author
|
Alrighty, this should be ready for review now. I've removed the truncation for the date fields and simplified them. We still need truncation for the filename in a DataViews context so I've tried to settle on a happy medium 🤞 |
ramonjd
approved these changes
Jun 3, 2026
Member
ramonjd
left a comment
ramonjd
left a comment
There was a problem hiding this comment.
LGTM behavior lines up with the PR intent IMO
Keyboard navigation is consistent in my testing.
Contributor
Author
|
Thanks for reviewing! I'll merge this in, but happy to follow-up in a separate PR if there's further feedback from designers, too 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What?
Fixes #78791
In the Media Editor Modal: re-order the details fields so that the regular layout fields appear at the top, with readonly fields at the bottom.
Why?
As raised in the issue, the editable fields can be more relevant to a user than the readonly / metadata ones. So this PR tries out re-ordering them, while also simplifying the Date added and modified fields so that they no longer truncate and simply use the
datetimefield type as-is.How?
text-wrap: pretty;so that we don't have widows when the date wrapsTesting Instructions
Screenshots or screencast
When the date and file name truncate to an extra row:
When there's a really long filename:
Use of AI Tools
Claude Code