WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#8886 closed enhancement (fixed)

Suggested UI changes for the edit media page

Reported by: designsimply Owned by: Nicholas91
Priority: low Milestone: 2.9
Component: Media Version: 2.7
Severity: normal Keywords: has-patch tested needs-review
Cc: dragos.nicholas@…

Description

Suggested UI changes to the edit media page shown at /wp-admin/media.php?action=edit&attachment_id=1

1. Add labels to info. at the top, i.e. File name, File type, Upload date

2. Add File URL field.

http://img.skitch.com/20090119-jufi8n1htyy3cr56u3gxiy33y1.png

Attachments (1)

media.diff (1011 bytes) - added by Nicholas91 4 years ago.
new labels with the new 'media-item-info-labels' class

Download all attachments as: .zip

Change History (12)

comment:1 mrmist4 years ago

  • Keywords needs-patch added
  • Version set to 2.7

comment:2 janeforshort4 years ago

Adding these labels and URL would be helpful to users. +1

comment:3 federico.bond4 years ago

Is there any predefined class we can use to highlight the labels as you do on your screen mockup? I don't think it would be correct to hard code orange or any other color in the template. Can we use just plain bold text?

Anyone has any idea how the File URL field functionality should be implemented? Is there a reason to make it editable, or should it appear below upload date as just a property?

comment:4 Nicholas914 years ago

  • Cc dragos.nicholas@… added
  • Keywords has-patch tested added; needs-patch removed
  • Owner set to Nicholas91

I've added the requested labels for the info at the top. These labels have a new class named 'media-item-info-labels'. Please tell me if you want them to be in just plain text or bold text, but I think adding a new class for them will be useful in the long run. Maybe a core dev can add the 'media-item-info-labels' class to the CSS files, thanks.

Nicholas914 years ago

new labels with the new 'media-item-info-labels' class

comment:5 Nicholas914 years ago

  • Keywords dev-feedback added

comment:6 Nicholas914 years ago

  • Keywords media removed

comment:7 follow-up: federico.bond4 years ago

I believe text must be in bold. I was not convinced about creating a new class though. But as you've said, it will probably be better in the long run.

comment:8 in reply to: ↑ 7 Nicholas914 years ago

Replying to federico.bond:

I believe text must be in bold. I was not convinced about creating a new class though. But as you've said, it will probably be better in the long run.

Thanks for your input, let's see what the devs decide.

comment:9 ShaneF4 years ago

  • Component changed from UI to Media
  • Milestone changed from 2.8 to 2.9

Moving all media items to 2.9.

comment:10 Denis-de-Bernardy4 years ago

  • Keywords needs-review added; dev-feedback removed

comment:11 azaozz4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [12195]) Add labels to image info in the media library, fixes #8886

Note: See TracTickets for help on using tickets.