Hi

I have created a submodule, which uses an iframe as a popup.

Do you want to merge it? I was thinking about making a standalone contrib, but the 6.x version is dev, so I think it can be a better idea to merge because of the API changes.

CommentFileSizeAuthor
imagecrop_iframe.tar_.gz2.01 KBtamasd

Comments

nils.destoop’s picture

nils.destoop commented

I installed it. But it doesn't look really user friendly. Next to my imagefield, i get 8 iframes below eachother (and i don't see for what preset). When i have multiple imagefields, it will be way more.
I think popups / overlays are good enough for imagecrop.

nils.destoop’s picture

nils.destoop commented
Status: Active » Closed (works as designed)
kvantomme’s picture

kvantomme commented
Status: Closed (works as designed) » Active

It is meant to have a setting in the content type which images caches are applicable to a certain content type. So the site might have 11 imagecaches, but only 2 are active on a given content type. That way the iframe approach is way more user friendly.

Would you reconsider in that case?

nils.destoop’s picture

nils.destoop commented

The problem is, from the moment that there are more then 1 or 2 iframes on the page. The cropping isn't that user friendly anymore, because you loose the overview.

nils.destoop’s picture

nils.destoop commented
Status: Active » Postponed
nils.destoop’s picture

nils.destoop commented
Version: 6.x-1.x-dev » 7.x-1.x-dev

This will be added in 2nd release of D7 version

nils.destoop’s picture

nils.destoop commented
Status: Postponed » Active
nils.destoop’s picture

nils.destoop commented
Status: Active » Fixed

Iframe support has been added to dev. Also to possibility to skip preview, and start cropping immediately

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.