keystone-classic icon indicating copy to clipboard operation
keystone-classic copied to clipboard

Cloudinary image gallery : labels and reordering

Open glimow opened this issue 5 years ago • 6 comments

I have used keystone in production and my customers are pretty happy with it. However, a significant part of them expected the ability to reorder images in the cloudinary gallery and to add label on them. I patched this (in a pretty dirty way) on my keystone, but I think that would be an interesting supported feature as most CMSs offer them.

glimow avatar Sep 25 '18 17:09 glimow

@glimow, could you tell us about your "dirty approach"? I'm very intrigued.

Antekirta avatar Jan 29 '19 09:01 Antekirta

Added fields in the field type and in the frontend component by hand in keystone code. Nothing is tested though

glimow avatar Feb 01 '19 16:02 glimow

@glimow I did similar approach on https://stanlywilson.me/work

It doesn't need any hard coding. Assigned a tag property to each image and a priority number to each image and sorted it out based on a category.

It would be quite useful if keystone had something like that in it build it.

abhijithvijayan avatar Apr 21 '19 13:04 abhijithvijayan

@abhijithvijayan I'm not sure I fully understand, You didn't do any any modification to keystone's code to achieve that ? That means you have not used the CloudinaryImages field but rather a List of CloudinaryImage along with other fields, am I wrong ?

glimow avatar Apr 21 '19 13:04 glimow

@glimow oh sorry, I didn't notice the CloudinaryImage(s) field, rather I used like you said a list of CloudinaryImage fields.

but the gallery view seems way better than my approach. Do you have a demo or something so as to check it out.

Thanks.

abhijithvijayan avatar Apr 21 '19 14:04 abhijithvijayan

Yeah the gallery is pretty cool, especially because I had to integrate a per-page gallery in my project. Unfortunately my code is a dirty hack and It's not very suitable for a demo or smth. But I'm up helping integrate that into keystone. The true problem is that keystone doesn't support nested lists, but I think this is not in the roadmap because there are other priorities.

glimow avatar Apr 22 '19 08:04 glimow