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

provide tile to include existing images from Plone #241

Closed
MrTango opened this Issue Aug 29, 2016 · 6 comments

Comments

Projects
None yet
4 participants
@MrTango

MrTango commented Aug 29, 2016

...instead of uploading a new one from the local hard-drive.
Eventually we can use also the "existing content" tile and integrate image including scaling in it.

@MrTango MrTango added this to the Mephisto Sprint 2016 milestone Aug 29, 2016

@agitator

This comment has been minimized.

Member

agitator commented Aug 30, 2016

or extend the already available image tile

@vangheem

This comment has been minimized.

Member

vangheem commented Sep 5, 2016

Maybe it's better to remove the current image tile implementation and instead advise users to use the image button on tinymce?

@zopyx

This comment has been minimized.

Member

zopyx commented Sep 5, 2016

What you actually want is an image title where you could select multiple existing images. As a display option you can choose from horizontal/vertical mode in order display images as a row or as a column or as a vision: render all selected images as a slider...

@vangheem

This comment has been minimized.

Member

vangheem commented Sep 5, 2016

Maybe that is more for the listing tile?

Perhaps the listing tile should allow you to select specific items. Then one of the displays for it would be "gallery" or "slider", etc.

However, typically with galleries and sliders you want other configuration options so perhaps that use-case should be handled as it's own tile.

@zopyx

This comment has been minimized.

Member

zopyx commented Sep 5, 2016

Possibly yes, depends on the assumption that a tile would imply always a fixed rendering method.

@vangheem

This comment has been minimized.

Member

vangheem commented Sep 7, 2016

I'm going to close this for now. To include existing images in plone, we advise to use tinymce to select and insert images.

@vangheem vangheem closed this Sep 7, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment