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

Adding the possibility to pick a specified size #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adding the possibility to pick a specified size #104

wants to merge 1 commit into from

Conversation

s43
Copy link

@s43 s43 commented Jan 26, 2015

Basically the variable "attachment.url" gives in return the full size of the image, I've been through a case where I needed to pick a specific cropped size, otherwise the images shows up in a bad resolution on the front-end.

The Wp media editor gives in return two arguments, props has an object with the selected size, and attachment has almost everything about the file, including the sizes with it urls.

I fixed it for my self, and I wanted to share it with others, and hopefully it may be helpful :).

Thanks for the good work guys, keep it up!

Basically the variable "attachment.url" gives in return the full size of the image, I've been through a case where I needed to pick a specific cropped size, otherwise the images shows up in a bad resolution on the front-end.

The Wp media editor returns gives is to arguments, props has an object with the selected size it, and attachment has almost everything about the file, including the sizes with it urls.

I fixed it for my self, and I wanted to share it with others, and hopefully it may be helpful :).

Thanks for the good work guys, keep it up!
@scrobbleme
Copy link

scrobbleme commented Sep 16, 2016

@s43 Maybe, you are interested to create a similar pull request for https://github.com/moewe-io/favpress.com

Thanks,

Adrian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants