Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Top Posts Widget: allow non-square image sizes #3682

Closed
jeherve opened this issue Apr 18, 2016 · 3 comments
Closed

Top Posts Widget: allow non-square image sizes #3682

jeherve opened this issue Apr 18, 2016 · 3 comments
Labels
[Feature] Extra Sidebar Widgets [Feature] Stats Data Feature that enables users to track their site's traffic and gain insights on popular content. Good For Community [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Type] Good First Bug

Comments

@jeherve
Copy link
Member

jeherve commented Apr 18, 2016

As explained in this tutorial, the jetpack_top_posts_widget_image_options filter allows one to change the size of the image used in the Top Posts widget.

Unfortunately, $get_image_options['avatar_size'] only accepts a single integer, and a square image is built from that value.

It would be nice if one could define an array of image width and image height instead, so the Top Posts images could be rectangular.

Suggested here:
https://wordpress.org/support/topic/chnage-size-of-post-thumbnails-jetpack-popular-post-widget?replies=1&view=all

@jeherve jeherve added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Stats Data Feature that enables users to track their site's traffic and gain insights on popular content. [Feature] Extra Sidebar Widgets [Type] Good First Bug labels Apr 18, 2016
@jeherve jeherve added this to the Community milestone Apr 18, 2016
@jb510
Copy link
Contributor

jb510 commented Aug 25, 2016

Related: The image sizing option here: https://jetpack.com/2013/08/20/custom-thumbnails-top-posts-widget/ seems to only apply to the grid layout ('grid'), not the image list ('list') view. I swear I forked this widget 6 months ago to overcome that... but frankly this widget is such a mess I'd recommend starting from scratch.

@jeherve
Copy link
Member Author

jeherve commented Aug 25, 2016

@jb510 As per our Slack chat, the CSS on that post was indeed outdated. I've updated it; thanks for the warning and for the help!

@lamdayap
Copy link

2886292-t

@samhotchkiss samhotchkiss modified the milestones: Community, Not Currently Planned Feb 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Extra Sidebar Widgets [Feature] Stats Data Feature that enables users to track their site's traffic and gain insights on popular content. Good For Community [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Type] Good First Bug
Projects
None yet
Development

No branches or pull requests

4 participants