[wp-trac] [WordPress Trac] #21391: Move attachment editing (media.php) to use standard post type UI
WordPress Trac
wp-trac at lists.automattic.com
Wed Sep 5 21:22:35 UTC 2012
#21391: Move attachment editing (media.php) to use standard post type UI
----------------------------+------------------
Reporter: nacin | Owner:
Type: task (blessed) | Status: new
Priority: normal | Milestone: 3.5
Component: Media | Version:
Severity: normal | Resolution:
Keywords: |
----------------------------+------------------
Comment (by helenyhou):
Finally have a first pass: [attachment:21391.2.diff], worked on with
ocean90.
Does the following:
* Exposes the regular post edit screen for attachments, which comes with
goodies like visual editor for the description, slug editing (works!),
taxonomy metaboxes (works!), and any other metaboxes
* Adds the image thumbnail with in-place image editing, currently above
the title (not beautiful, but works!)
* Adds a metabox for editing alt text, caption (really the excerpt), and
displaying the file URL
* Introduces `disable_for_post_type()` and `post_type_disables()`.
Currently utilized in the publish metabox
* Adds hooks to edit_form_advanced.php
Known issues:
* "Trash" in the publish metabox should really be "Delete Permanently" (or
rather, based on the MEDIA_TRASH internal constant like it is elsewhere)
* Alt text shows but doesn't actually save
* The layout of the screen isn't awesome, at all. In particular, the
placement of the image+details+editor at the top doesn't look great with
the publish metabox still at the top of the side metaboxes. Also, it does
feel a little clunky to have the title and description edited separately
from the other pieces of data, when they were formerly indicated as being
essentially equal in importance, and realistically the alt text and
caption are perhaps more important.
Things to think about:
* What happens when you add new media via media-new.php? Upload and then
punt over to editing it immediately? Something else?
* Do we want to expose attached status (parent post) on the screen
somewhere? It is on the list table, so it's not lost.
* Should we leave the list table where it is (a special upload.php screen)
or move to the `/wp-admin/edit.php?post_type=attachment` screen instead?
I'm sure there's more. :)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/21391#comment:27>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list