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

Provide a separate Image tile instead of using Richtext tile #104

Open
MrTango opened this issue Jul 4, 2019 · 5 comments
Open

Provide a separate Image tile instead of using Richtext tile #104

MrTango opened this issue Jul 4, 2019 · 5 comments

Comments

@MrTango
Copy link
Contributor

MrTango commented Jul 4, 2019

I realized that the old image tile way has some advantages over using the Richtext tile, especially for pre defined content layouts, where you want to have images as place holders and want to allow the user just to change the image but not the size or the location of the image. With the richtext tile we can prevent the moving but can not have pre defined fixed sizes. That would be really help full. I can imagine that it should be possible to create an Image tile, which uses the upload pattern to upload an image to the current context, either next to a document or inside it, if it is a folderish type.
The tile should allow to set the meta data like the description and so on as well es pre define a scale for the image. The mosiac content layout should be able to disallow changing the scale.

@datakurre
Copy link
Member

I believe that the code for separate image tiles is still there. The issue with it was that it required duplicating much of the scaling code in plone.namedfile and that code should be manually kept in sync with plone.namedfile changes. Also image tile blobs are not versioned yet.

@agitator
Copy link
Member

agitator commented Jul 5, 2019

@MrTango it would be handy to have an image tile and extend it with a target link (and optional override of title and description) to be used as a teaser tile.

@datakurre
Copy link
Member

We are using in-house tiles with similar features, but with uuid-linked image instead of image saved with with tile. It was not as convenient as images saved in tile data for just images, but has been working pretty well with leadimage-behaviored content.

@mauritsvanrees
Copy link
Member

I believe that the code for separate image tiles is still there.

True. But @jensens is removing it for Plone 6.

The issue with it was that it required duplicating much of the scaling code in plone.namedfile and that code should be manually kept in sync with plone.namedfile changes.

True. But I have recently made it easier to customise or override parts of plone.namedfile. Latest plone.app.tiles alpha uses this. The remaining customisations are really only about getting data from the tile instead of the content item. So I believe this reason does not count anymore.

Also image tile blobs are not versioned yet.

I have not checked, but this is probably still true.

@jensens
Copy link
Member

jensens commented Mar 21, 2022

Idea is to have a new tile referencing a real image content (including direkt upload from reference field). Otherwise the alt-text/title/copyright hell kicks in.

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

No branches or pull requests

5 participants