paperclip.additional_thumbnails isn’t respected

[gerrit]

While its mentioned in the README, it doesn’t have any effect (it isn’t accessed anywhere in the code). Is this intentional? If so, what is the replacement method to specify custom sizes?

are assets.thumbnails.image, assets.thumbnails.video and assets.thumbnails.pdf what you're looking for?

[gerrit]

Yes, thats it. I'll update the docs soon.
There is support for a legacy option called “assets.additional_thumbnails”. Was that ever in use? If so we'd need to support both legacy options

looks like additional_thumbnails was part of the original paperclipped extension. http://git.io/8-XYHA#L161-163

[gerrit]

whoops, wrong issue number in commit

[gerrit]

Ok so even paperclipped never used paperclip.additional_thumbnails.
I think the real issue here is that the documentation states to use paperclip.additional_thumbnails when that never worked. I might be the only one that followed the docs, but I still think we should handle that gracefully. I'll patch the legacy support code too if that’s ok

[lelesrc]

We followed the doc too...

but paperclip.additional_thumbnails never did anything right? so by aliasing it to assets.thumbnails.image we'll actually be changing behavior right? i'm fine with that since doing nothing was obviously not the right thing; just trying to figure out what behavior was expected. (yeah i never really used the paperclipped extension or read the clipped docs so sorry if this is a stupid question)

[lelesrc]

is there a known workaround for this issue?
basically, we just need to add "strip=180x100>" to our thumbnails list...

@youleaf the workaround is to use the real config settings: assets.thumbnails.image, assets.thumbnails.video and assets.thumbnails.pdf

the fix is to update the docs and possibly alias paperclip.additional_thumbnails to assets.thumbnails.image

[saturnflyer]

@johnmuhl updated this extension to add clipped_config.rb

Is this still an issue?

we still need some changes to the readme to remove references to stuff like this.