search

MichaelArestad / Press-This

Posting images, links, and cat gifs will never be the same.
23 stars 11 forks source link

Should we sideload the full size image? #82

Open MichaelArestad opened 9 years ago

MichaelArestad commented 9 years ago

What's not as cool is teh full size sideloaded image. I'd change that every post if I was using this, to at least use "large"

via @krogsgard

stephdau commented 9 years ago

Is that suggestion to just apply a standard wp class name to the img tag we insert in the editor?

krogsgard commented 9 years ago

No, I mean to not insert the full size image. If it's coming from another website it could be quite large. Like 2000px or something. Once it sideloads and inserts, I'd insert it at "large" bc that should reasonably be the largest inserted image for any WordPress site, per their media settings.

Brian Krogsgard Krogsgard@gmail.com Krogsgard.com 251.709.2224

On Fri, Feb 20, 2015 at 11:44 AM, Stephane Daury notifications@github.com wrote:

Is that suggestion to just apply a standard wp class name to the img tag we insert in the editor?

— Reply to this email directly or view it on GitHub https://github.com/MichaelArestad/Press-This/issues/82#issuecomment-75285420 .

DrewAPicture commented 9 years ago

I agree with @krogsgard. Sideload the full-size image, sure, but then only insert the 'large' image size. We'd probably have to make that image size filterable though.