WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 8 years ago

#8886 closed enhancement (fixed)

Suggested UI changes for the edit media page

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

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 8 years ago.
new labels with the new 'media-item-info-labels' class

Download all attachments as: .zip

Change History (12)

#1 @mrmist
9 years ago

  • Keywords needs-patch added
  • Version set to 2.7

#2 @janeforshort
9 years ago

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

#3 @federico.bond
9 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?

#4 @Nicholas91
8 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.

@Nicholas91
8 years ago

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

#5 @Nicholas91
8 years ago

  • Keywords dev-feedback added

#6 @Nicholas91
8 years ago

  • Keywords media removed

#7 follow-up: @federico.bond
8 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.

#8 in reply to: ↑ 7 @Nicholas91
8 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.

#9 @ShaneF
8 years ago

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

Moving all media items to 2.9.

#10 @Denis-de-Bernardy
8 years ago

  • Keywords needs-review added; dev-feedback removed

#11 @azaozz
8 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.