Ticket #5911 (closed defect: fixed)

Opened 5 months ago

Last modified 2 months ago

Media Library updates

Reported by: andy Assigned to: ryan
Priority: normal Milestone: 2.5
Component: Administration Version: 2.5
Severity: normal Keywords: media has-patch
Cc:

Description

Target: bring the Media Library page into line with the Happy Cog mockup.

Patches: coming in bursts.

Attachments

5911-1.diff (24.8 kB) - added by andy on 02/19/08 05:07:06.
5911-2.diff (0.5 kB) - added by andy on 02/20/08 17:00:10.
Fix the Delete button AYS
5911-3.diff (49.9 kB) - added by andy on 02/22/08 05:25:03.
images.zip (26.9 kB) - added by andy on 02/22/08 05:41:43.
file icons for wp-includes/images
5911-4.diff (3.7 kB) - added by andy on 02/26/08 05:51:00.
update mime-to-icon matching
5911-5.diff (0.6 kB) - added by andy on 02/26/08 06:00:08.
get filename from function instead of guid. fixes h-scroll caused by bad guid.
5911-6.diff (22.9 kB) - added by andy on 02/26/08 11:53:06.
incremental changes to bring behavior up to latest wireframes
5911-7.diff (484 bytes) - added by andy on 02/26/08 19:31:36.
fix overuse of default mime icon
5911-8.diff (0.6 kB) - added by andy on 02/26/08 19:40:44.
re-shuffle mp4/m4a file type matches
5911-9.diff (9.1 kB) - added by andy on 02/27/08 08:21:05.
first pass at Media Library uploader tab
5911-10.diff (489 bytes) - added by andy on 02/27/08 18:48:17.
remove debug cruft
5911-11.diff (0.6 kB) - added by andy on 02/27/08 19:19:06.
set swfupload max file size to match php ini
5911-12.diff (36.6 kB) - added by andy on 02/28/08 20:59:23.
Button Revival!
5911-13.diff (7.7 kB) - added by andy on 02/29/08 05:09:09.
error improvements, localization, and icon improvements -- please do not commit before new icons arrive in wp-includes/images/crystal
5911-14.diff (433 bytes) - added by andy on 03/03/08 19:53:59.
Fix table styling
5911-15.diff (0.9 kB) - added by andy on 03/03/08 23:56:28.
Revive and filter "Add media:" text beside upload buttons
5911-16.diff (7.6 kB) - added by andy on 03/05/08 06:30:08.
Improve upload_max_filesize error. Improve error display for long messages. New uploads now added to bottom, not top. Fix align class. Remove redundant header text.
5911-17.diff (9.2 kB) - added by andy on 03/06/08 06:23:34.
Fix insert, gallery, sizing, type queries, errors, thumbs, type fallback.
5911-18.diff (452 bytes) - added by andy on 03/06/08 23:16:29.
fix missing thumbnail radio button for inserting into post
5911-19.diff (0.6 kB) - added by andy on 03/07/08 19:43:10.
use new colors stylesheet
5911-20.diff (2.4 kB) - added by andy on 03/10/08 21:26:31.
fix icons
5911-21.diff (1.3 kB) - added by andy on 03/10/08 21:46:09.
fix invisible delete links (now with media-upload iframe body id)
5911-22.diff (1.5 kB) - added by andy on 03/10/08 22:50:35.
more thumbnail fixes
5911-23.diff (1.2 kB) - added by andy on 03/10/08 23:06:38.
remove ugliness from default attachment template
5911-24.diff (6.5 kB) - added by andy on 03/11/08 00:03:54.
5911-25.diff (1.2 kB) - added by andy on 03/11/08 00:17:50.
translate "Crunching..."
5911-26.diff (4.2 kB) - added by andy on 03/11/08 01:12:20.
add Save button and fix tab+type breakage
5911-27.diff (5.4 kB) - added by andy on 03/11/08 21:53:19.
"Deleted" message
5911-28.diff (5.8 kB) - added by andy on 03/12/08 07:58:29.
use array(width, height) sizes in new image funcs
5911-29.diff (18.9 kB) - added by andy on 03/17/08 19:48:07.
last patch on this ticket: remove obviously unreachable code
default-media-icon-size.png (22.9 kB) - added by iammattthomas on 03/18/08 21:16:35.
Showing bad effects of min-height: 40px; on the thumbnail/icon
5911-rel.diff (2.6 kB) - added by andy on 03/21/08 22:57:46.
add rel attrs to attachment links inserted from uploader
5911-delpretty.diff (4.0 kB) - added by andy on 03/21/08 23:10:20.
pretty delete animation
5911-iconpretty.diff (0.8 kB) - added by andy on 03/21/08 23:13:27.
fix squashed icons and use 80x60 thumbs
5911-master.diff (4.8 kB) - added by andy on 03/21/08 23:18:52.

Change History

02/19/08 05:07:06 changed by andy

  • attachment 5911-1.diff added.

02/19/08 06:13:21 changed by ryan

(In [6910]) Media Library design updates from Andy. see #5911

02/19/08 06:13:57 changed by ryan

  • owner changed from ryan to andy.

02/20/08 17:00:10 changed by andy

  • attachment 5911-2.diff added.

Fix the Delete button AYS

02/20/08 17:00:32 changed by andy

  • owner changed from andy to ryan.

02/20/08 17:05:43 changed by ryan

(In [6941]) Fix media delete nonce. Props andy. see #5911

02/22/08 05:25:03 changed by andy

  • attachment 5911-3.diff added.

02/22/08 05:41:43 changed by andy

  • attachment images.zip added.

file icons for wp-includes/images

02/22/08 05:53:48 changed by ryan

(In [6974]) Media library work from andy. see #5911

02/26/08 05:51:00 changed by andy

  • attachment 5911-4.diff added.

update mime-to-icon matching

02/26/08 06:00:08 changed by andy

  • attachment 5911-5.diff added.

get filename from function instead of guid. fixes h-scroll caused by bad guid.

02/26/08 08:54:11 changed by ryan

(In [7036]) update mime-to-icon matching and get filename from function instead of guid. Props andy. see #5911

02/26/08 11:53:06 changed by andy

  • attachment 5911-6.diff added.

incremental changes to bring behavior up to latest wireframes

02/26/08 19:30:10 changed by ryan

(In [7043]) Media upload updates from andy. see #5911

02/26/08 19:31:36 changed by andy

  • attachment 5911-7.diff added.

fix overuse of default mime icon

02/26/08 19:40:44 changed by andy

  • attachment 5911-8.diff added.

re-shuffle mp4/m4a file type matches

02/26/08 19:57:00 changed by ryan

(In [7045]) fix overuse of default mime icon and re-shuffle mp4/m4a file type matches. Props andy. see #5911

02/27/08 08:21:05 changed by andy

  • attachment 5911-9.diff added.

first pass at Media Library uploader tab

02/27/08 08:31:10 changed by ryan

(In [7062]) Add media library uploader tab. Props andy. see #5911

02/27/08 18:48:17 changed by andy

  • attachment 5911-10.diff added.

remove debug cruft

02/27/08 19:19:06 changed by andy

  • attachment 5911-11.diff added.

set swfupload max file size to match php ini

02/27/08 19:36:03 changed by ryan

(In [7067]) remove debug cruft and set swfupload max file size to match php ini. Props andy. see #5911

02/28/08 20:59:23 changed by andy

  • attachment 5911-12.diff added.

Button Revival!

02/28/08 21:29:52 changed by ryan

(In [7092]) Media library updates from andy. see #5911

02/29/08 05:09:09 changed by andy

  • attachment 5911-13.diff added.

error improvements, localization, and icon improvements -- please do not commit before new icons arrive in wp-includes/images/crystal

02/29/08 06:54:15 changed by ryan

(In [7100]) Media Library updates from andy. see #5911

03/03/08 19:53:59 changed by andy

  • attachment 5911-14.diff added.

Fix table styling

03/03/08 23:56:28 changed by andy

  • attachment 5911-15.diff added.

Revive and filter "Add media:" text beside upload buttons

03/04/08 04:28:24 changed by matt

(In [7151]) Table styling and "add media" label. See #5911. Hat tip: andy.

03/05/08 06:30:08 changed by andy

  • attachment 5911-16.diff added.

Improve upload_max_filesize error. Improve error display for long messages. New uploads now added to bottom, not top. Fix align class. Remove redundant header text.

03/05/08 06:46:42 changed by ryan

(In [7160]) Improve upload_max_filesize error. Improve error display for long messages. New uploads now added to bottom, not top. Fix align class. Remove redundant header text. Props andy. see #5911

03/06/08 06:23:34 changed by andy

  • attachment 5911-17.diff added.

Fix insert, gallery, sizing, type queries, errors, thumbs, type fallback.

03/06/08 19:48:55 changed by ryan

(In [7172]) Fix insert, gallery, sizing, type queries, errors, thumbs, type fallback. Props andy. see #5911

03/06/08 23:16:29 changed by andy

  • attachment 5911-18.diff added.

fix missing thumbnail radio button for inserting into post

03/06/08 23:40:22 changed by ryan

(In [7177]) fix missing thumbnail radio button for inserting into post. Props andy. see #5911

03/07/08 14:56:30 changed by westi

  • severity changed from major to blocker.

Making a release blocker - we should close this when we are happy with the media library.

03/07/08 19:43:10 changed by andy

  • attachment 5911-19.diff added.

use new colors stylesheet

03/07/08 22:40:08 changed by westi

(In [7185]) Use the new colours stylesheet for the media library. See #5911 props andy.

03/10/08 21:26:31 changed by andy

  • attachment 5911-20.diff added.

fix icons

03/10/08 21:31:33 changed by matt

(In [7212]) Icon fixes. See #5911; hat tip: andy.

03/10/08 21:46:09 changed by andy

  • attachment 5911-21.diff added.

fix invisible delete links (now with media-upload iframe body id)

03/10/08 21:49:09 changed by matt

(In [7213]) Invisible ink delete links. See #5911. Also some old school colors, half-way done.

03/10/08 22:50:35 changed by andy

  • attachment 5911-22.diff added.

more thumbnail fixes

03/10/08 23:04:21 changed by matt

(In [7218]) More attachment thumbnail fixes, see #5911. Hat tip: andy.

03/10/08 23:06:38 changed by andy

  • attachment 5911-23.diff added.

remove ugliness from default attachment template

03/11/08 00:03:54 changed by andy

  • attachment 5911-24.diff added.

03/11/08 00:09:15 changed by matt

(In [7222]) Better media attachment template. See #5911. Hat tip: andy.

03/11/08 00:17:50 changed by andy

  • attachment 5911-25.diff added.

translate "Crunching..."

03/11/08 01:12:20 changed by andy

  • attachment 5911-26.diff added.

add Save button and fix tab+type breakage

03/11/08 01:30:17 changed by matt

(In [7228]) See #5911. Hat tip: andy.

03/11/08 21:53:19 changed by andy

  • attachment 5911-27.diff added.

"Deleted" message

03/11/08 23:15:18 changed by ryan

(In [7256]) media handling fixes from andy. see #5911

03/12/08 07:58:29 changed by andy

  • attachment 5911-28.diff added.

use array(width, height) sizes in new image funcs

03/12/08 08:10:00 changed by ryan

(In [7263]) Image fixes from andy. see #5911

03/14/08 20:24:36 changed by iammattthomas

I just noticed this, but patch 5911-14.diff in [7151] squishes the media icons (60px tall) into a max-height of 40px.

If it's necessary for image thumbnails, a class should be applied to thumbnails that gives them the max-height. Or, a class could be applied to file icons that gives them a height of 60px.

03/15/08 22:27:30 changed by westi

Now we have a funky media library that works a dream do we still need to keep wp-admin/includes/upload.php as it's not referenced by any WordPress code.

If so I guess we should make it as deprecated so we can remove it later.

03/16/08 15:07:27 changed by KirkM

I'm hoping this is the correct place for this. In my testing I've found a problem with the current Media Library. It seems to have been around for the last few revisions. I've compared two different test sites below that exhibit the same problem.

In the "Manage/MediaLibrary", when selecting any link in the "Media" column brings up the selected image's edit page as usual. At that point "Settings", "Plugins" and "Users" disappears from the top right of page and the following warning is displayed at the bottom of the page:

Warning*: Cannot modify header information - headers already sent by (output started at /home/longmead/public_html/just-thinkin/wp-admin/admin-header.php:26) in */home/longmead/public_html/just-thinkin/wp-includes/pluggable.php* on line *688

All plugins are deactivated during this test.

On my other test site currently running on r2998 the same thing occurs with the exception of slight change in the warning (different line #):

*Warning*: Cannot modify header information - headers already sent by (output started at /home/longmead/public_html/just-thinkin/wp25-test/wp-admin/admin-header.php:26) in */home/longmead/public_html/just-thinkin/wp25-test/wp-includes/pluggable.php* on line *687

Again, no plugins activated.

I'm to light yet on the ins and outs of the Wordpress code to troubleshoot this myself.

(follow-up: ↓ 30 ) 03/16/08 15:09:03 changed by KirkM

Correction to my last comment. In the first example, that site is running r7331.

(in reply to: ↑ 29 ) 03/16/08 17:26:08 changed by KirkM

Replying to KirkM:

Correction to my last comment. In the first example, that site is running trunk build r7331 and in the second example the trunk build is r7298.

This may be related to ticket #6206

03/17/08 19:48:07 changed by andy

  • attachment 5911-29.diff added.

last patch on this ticket: remove obviously unreachable code

03/17/08 20:07:08 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [7347]) Remove old upload files. props andy. fixes #5911

03/17/08 22:48:42 changed by bomberstudios

  • priority changed from high to normal.
  • status changed from closed to reopened.
  • resolution deleted.
  • severity changed from blocker to normal.
  • type changed from enhancement to defect.

As of r7355, upload.php is still linked from some pages in the admin area (i.e: Manage » Media Library)

I'm reopening the ticket in the hopes that either this file comes back or the link in edit.php is updated to whatever replaces upload.php

03/17/08 22:53:14 changed by westi

(In [7356]) Restore upload.php as we really need it. See #5911.

03/17/08 22:54:13 changed by westi

(In [7357]) Delete the correct upload.php. See #5911 props andy.

03/17/08 22:56:46 changed by westi

  • status changed from reopened to closed.
  • resolution set to fixed.

done

03/18/08 21:16:35 changed by iammattthomas

  • attachment default-media-icon-size.png added.

Showing bad effects of min-height: 40px; on the thumbnail/icon

03/18/08 21:18:23 changed by iammattthomas

  • status changed from closed to reopened.
  • resolution deleted.

It may have been overlooked, but I mentioned previously that the min-height of 40px needs to be adjusted as it's unnaturally squashing the media library icons. See screenshot.

03/21/08 22:57:46 changed by andy

  • attachment 5911-rel.diff added.

add rel attrs to attachment links inserted from uploader

03/21/08 23:10:20 changed by andy

  • attachment 5911-delpretty.diff added.

pretty delete animation

03/21/08 23:13:27 changed by andy

  • attachment 5911-iconpretty.diff added.

fix squashed icons and use 80x60 thumbs

03/21/08 23:18:52 changed by andy

  • attachment 5911-master.diff added.

03/21/08 23:21:28 changed by ryan

(In [7466]) pretty delete animation, fix squashed icons and use 80x60 thumbs, add rel attrs to attachment links inserted from uploader. Props andy. see #5911

03/25/08 01:59:36 changed by technosailor

For a brief few hours on Saturday, I was pleased to see a tags field added to the attachment editing page. It's gone now and I keep hoping it will come back. Can we get that back? Please?

03/28/08 11:11:30 changed by DD32

A comment on a blog post of mine:

also if I go to manage => media library then below “appears with” => it says: “no title” I click on no title, assuming I could edit its title, and get this error:

You attempted to edit a post that doesn’t exist. Perhaps it was deleted?

the url changes to: http://pacura.ru/wordpress/wp-admin/post.php?action=edit&post=-1206700048

it seems the media library tries to edit a post?

I can reproduce it on the latest Trunk too.

Create a new draft, attach an image, make sure it saves, Head over to the Manage Media library, the post link on the Image "Appears with" directs to the Draft post ID, not the real post id.

04/21/08 17:38:00 changed by ryan

  • status changed from reopened to closed.
  • resolution set to fixed.
  • milestone changed from 2.6 to 2.5.