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

Add option to show thumbnails for images in Plone 5 content views #1074

Closed
fgrcon opened this issue Sep 24, 2015 · 3 comments
Closed

Add option to show thumbnails for images in Plone 5 content views #1074

fgrcon opened this issue Sep 24, 2015 · 3 comments

Comments

@fgrcon
Copy link
Member

fgrcon commented Sep 24, 2015

This is more or less deprecated and solved by other means see coment down at the bottom

image

image

rename

Affected components:

  • mockup pattern structure
  • plone.app.content

requires fix #1129 (plone.app.dexterity, ploneapp.contenttypes)

@fgrcon
Copy link
Member Author

fgrcon commented Oct 29, 2015

@vangheem @jensens ....:
After cleaning up all templates (as far i could spot hem all) I tried to unify everything as much as possible. Thumbs are shown in all listings whe item is an Image or item has a leadimage. This also applies to folder contents in patt structure as well. So there is no need to extend the vocubalary any more and available Column Thumb is not needed anymore either.
foldercontents

The only important change is in p.a.contenttypes: redefined metadata field: getIcon.
I suppose this will require an upgrade step to reindex?

We still have a small problem: we need to replace portal_type.toLowerCase() with a js function that more or less implemets normalizeString (e.g: 'news Item' --> 'news-item' also important for custom types). I tried a lot but got totally lost in the js-stuff .

@hvelarde
Copy link
Member

awesome, adding the Description to the batch rename would be an appreciated extra.

@agnogueira @rodfersou what do you think?

@davilima6
Copy link
Member

Very nice addition indeed! I'd only align image right as in news items, even if it means narrower field widths for some items.

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

4 participants