plone / plone.app.standardtiles

Plone Standard tiles (reflecting viewlets et al) to be used with Plone Mosaic
Other
7 stars 12 forks source link

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

Open MrTango opened 5 years ago

MrTango commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

@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 commented 5 years ago

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 commented 2 years ago

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 commented 2 years ago

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.