Add feature image icon to admin post list

[christopherross]

A simple patch to add a copy of the feature image to the admin's post list if one exists.

[dd32]

You may want to review the changes there, Most seem to be reverting changes made in 3.2 rather than adding the feature you intended.

Also, when submitting patches, only include the .dev versions of css and js files, the commitor will re-minify it afterwards.

[christopherross]

thanks for the tips, I've not submitted a patch before so I really appreciate it. I'll take a look as you've suggested.

[nacin]

In [18426]:

Check for WP_Error return values when using get_term_field() to set up nav menu items. fixes #17958 for trunk.

[nacin]

In [18427]:

Check for WP_Error return values when using get_term_field() to set up nav menu items. fixes #17958 for 3.2.

[nacin]

Oops, those belonged on #17968.

[christopherross]

new diff for patch to add icon to admin

[christopherross]

I know this patch isn't a high priority or even likely to be included without substantial consideration but it's my first attempt to contribute to the core and would very much appreciate feedback to know if the new patch file is done right, for future reference.

[nacin]

That code could use a lot of spacing, but yes, patch is correct.

[mikeschroder]

Refresh of patch with additional spacing

[mikeschroder]

Image showing posts including their featured thumbs

[mikeschroder]

This includes the thumbs for pages as well, if it's desired.

[scribu]

That's a pretty neat idea.

I think it's a small-enough patch for 3.3.

[mikeschroder]

Thanks for the feedback!
Per your notes in IRC, I'm also going ahead and cleaning up the formatting in the file in general, since it most certainly can use it.

[christopherross]

minor spacing fix around the sizing array

[mikeschroder]

*only* reformatting of this specific section of the file, with no added icons

[mikeschroder]

Reformatting *and* featured post icons for pages and posts

[mikeschroder]

Attached files both with formatting only for this hairy section of code, and with the formatting+change.
I hope to also reformat more of the file (or the entire file), although if the conversion to API/CSS from tables for admin is done in 3.3, this will likely be duplicated effort.

[mikeschroder]

Refreshed with nacin's cleanup

[mikeschroder]

Created #18664 for patches/discussion on cleanup/formatting changes.

[chexee]

Aligns thumbnail with top of post title and bottom of action links if post title is one line

[chexee]

I really like this. Altered DH-Shredder's patch a bit so the thumbnail is aligned a bit better:

​http://chx.mx//1x/qne90hmy2skk4.png

From a user perspective, I think this would really do well to drive home what a featured image is v. including an image in a post. It represents that post on both the front-end and back-end of your site.

[chexee]

After a bit more testing, the alignment looks good even with multi-line posts.

Noticed it loads the entire 150x150 thumbnail, which might get slow for certain users if they have lots of posts with featured thumbs (even with resized images, might still get pretty slow).

[scribu]

Replying to chexee:

Noticed it loads the entire 150x150 thumbnail, which might get slow for certain users if they have lots of posts with featured thumbs (even with resized images, might still get pretty slow).

It actually depends on how many posts they want to display per screen. I don't think it's worth registering a whole new intermediate image size just for this.

[nacin]

Replying to scribu:

Replying to chexee:

Noticed it loads the entire 150x150 thumbnail, which might get slow for certain users if they have lots of posts with featured thumbs (even with resized images, might still get pretty slow).

It actually depends on how many posts they want to display per screen. I don't think it's worth registering a whole new intermediate image size just for this.

I don't think it's worth the extra 20 or more HTTP requests. Seems like a good plugin for now.

[scribu]

Agreed. Better not to force this on users.

[scribu]

And here is such a plugin:

​http://wordpress.org/extend/plugins/codepress-admin-columns/screenshots/