Closed (fixed)
Project:
Image
Version:
6.x-1.0-alpha6
Component:
image_gallery
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Aug 2009 at 03:24 UTC
Updated:
14 Sep 2009 at 17:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
Hetta commentedDo you have views installed? If so, what are the settings for the views image gallery?
Comment #2
deshilachado commentedsame problem here
after running update.php there were a lot of messages, no erros and now all galleries display ten images and no pager.
there are always nine images in three rows & three columns and then one image below, centered.
Comment #3
deshilachado commentedsorry, i didn't mean to change the status.
before you said it, i didn't know image is using views for the gallery
i went to http://example.com/admin/build/views/edit/image_gallery
and changed the items_per_page from 10 to 0 (unlimited)
this works for me, thanks a lot
Comment #4
deshilachado commentedbut stil it is strange that under
http://example.com/admin/settings/image/image_gallery
one can set a number of images per gallery but this settings doesn't change anything, as far as i can see
Comment #5
Hetta commentedThe settings under image_gallery work if you get rid of views. You can, if you so like, submit a patch adding some help text to the image_gallery admin page.
Comment #6
deshilachado commentedok, good to know.
in case anyone wants to read more about image and views: #405456: Galleries made with views
right now i'm at work, but if noone is faster than me i'll try to find the time this weekend to write that help text (allthough i know it will take me hours to re-learn how to write a patch...)
Comment #7
joachim commentedOk, to fix I would say we need:
- message on upgrade that galleries now made with views (can update functions use dsm?)
- note on the IG settings page. this can be cribbed from D5. Totally forgot about that!
I am away from teh nets till conference -- a patch would be great!
Comment #8
dugh commentedSee same issue
My images are all munged together overlapping. Will work with views
Comment #9
shelleyp commentedThe change also has caused problems with the display, and you have to remember to add paging to get beyond the first set of images.
All in all, a rather disruptive update. Can you give us some clues about what we need to do, both to the view and to the styles to make the display similar to what we had before this update?
Comment #10
joachim commentedIt should look about the same.
Though the note about paging -- yes, should have paging built in. Please file another bug for this!
Comment #11
joachim commented> All in all, a rather disruptive update.
Big changes yes, but this is image module getting to where it needs to be. Anything in an alpha release is subject to change :)
Comment #12
shelleyp commented"changes yes, but this is image module getting to where it needs to be. Anything in an alpha release is subject to change :)"
Oh sure, I understand. I'll find a bug for the paging, but it's easy enough to modify in the view.
I have found serious styling issues, too. Images overlapping each other in the gallery. Is there anything on differences in layout between the view and the original gallery that may be different?
Comment #13
joachim commentedCommitted the attached patch.
This adds a note to the gallery settings page to explain what's going on, and removes the settings while the view is enabled, as they will have no effect.
Thanks for reporting this :)