Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#22439 closed enhancement (fixed)

Remove "Attached To" column from the attachments table

Reported by: scribu's profile scribu Owned by: nacin's profile nacin
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: has-patch
Focuses: Cc:

Description (last modified by scribu)

Before 3.5, an attachment's parent post meant "this is the post this file belongs to; it will automatically show up in the gallery". In 3.5, if it's even set, it means "this is the post this file was originally uploaded to".

After just seeing a user trying to insert an image into a post using the "Attached To" column (around 2:30), I'm thinking maybe we should hide that column.

Attachments (3)

22439.diff (650 bytes) - added by scribu 12 years ago.
22439.2.diff (1.7 KB) - added by scribu 12 years ago.
22439.3.diff (685 bytes) - added by helenyhou 12 years ago.

Download all attachments as: .zip

Change History (10)

#1 @scribu
12 years ago

  • Description modified (diff)

@scribu
12 years ago

@scribu
12 years ago

#2 @scribu
12 years ago

  • Keywords has-patch added

The first patch just removes the column registration; the second removes the column implementation as well (the JS logic still remains).

#3 follow-up: @mdgl
12 years ago

See also #22085. It seems we need to think more carefully about what we're doing here.

Changing the gallery shortcode to rely only on a list of IDs is fine for posts created in 3.5 or later, but for existing posts we still need to be able to find gallery images by searching for attachment children. There is also the issue of pretty permalinks for these attached children, which appear as though they "belong" to the parent post.

Attachments also don't need to be either images or in a gallery and many sites/themes may be relying on grouping such content together through the post_parent field and use of APIs such as {wp_}get_attachment_link(), {next/previous/adjacent}_image_link and so on.

So if you are trying to preserve some kind of backwards compatibility in 3.5 I suspect the post that an attachment is, erm, "attached" to is still of interest/concern to users.

Finally, don't forget #21391 kinda promised some kind of support for reparenting attachments.

#4 @nacin
12 years ago

I'd like to leave this for 3.5, but rename it to "Uploaded to".

#5 in reply to: ↑ 3 @helenyhou
12 years ago

Replying to mdgl:

Finally, don't forget #21391 kinda promised some kind of support for reparenting attachments.

#21391 says

We can get a lot of things "for free" if we move it to be a proper post type UI, including: ... Re-parenting (even if only exposed by a plugin)

Last edited 12 years ago by helenyhou (previous) (diff)

@helenyhou
12 years ago

#6 @helenyhou
12 years ago

22439.3.diff to just rename it. I think it's better to retain the column for 3.5 - de-emphasizing doesn't necessarily warrant its removal right now.

Semi-related: #20164 to make that attach-to-something box better, which I guess would also disappear if the column did.

#7 @nacin
12 years ago

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

In 22630:

Media Library list table: Rename 'Attached to' to 'Uploaded to'.

In 3.5, we have chosen to de-emphasize the "attaching" part of the
post-attachment relationship. Existing functionality remains the same,
but UI emphasis is now placed on "parent" being where the item was
originally uploaded.

fixes #22439.

Note: See TracTickets for help on using tickets.