I've taken this one as far as I plan to...

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/img/upload.php?r1=1.31&r2=1.32&ty=u
http://cvs.horde.org/diff.php/ansel/img/upload_preview.php?r1=1.4&r2=1.5&ty=u
http://cvs.horde.org/diff.php/ansel/lib/Ansel.php?r1=1.546&r2=1.547&ty=u
http://cvs.horde.org/diff.php/ansel/templates/image/upload.inc?r1=1.7&r2=1.8&ty=u

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/img/upload_preview.php?r1=1.1.2.1&r2=1.1.2.2&ty=u
http://cvs.horde.org/diff.php/ansel/js/carousel.js?r1=1.1.2.1&r2=1.1.2.2&ty=u
http://cvs.horde.org/diff.php/ansel/js/src/carousel.js?r1=1.1.2.1&r2=1.1.2.2&ty=u
http://cvs.horde.org/diff.php/ansel/templates/image/upload.inc?r1=1.1.2.2&r2=1.1.2.3&ty=u

Alright. This now seems to be in pretty good shape. Still to come, 
image should open in a lightbox when clicked, some CSS issues to work 
out and we should probably have  start/end navigation in addition to 
the next/previous navigation.



Merging to FW_3 as well.

http://lists.horde.org/archives/cvs/Week-of-Mon-20081020/084705.html

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/img/upload_preview.php?r1=1.3&r2=1.4&ty=u
http://cvs.horde.org/diff.php/ansel/js/carousel.js?r1=1.1&r2=1.2&ty=u
http://cvs.horde.org/diff.php/ansel/js/src/carousel.js?r1=1.1&r2=1.2&ty=u
http://cvs.horde.org/diff.php/ansel/templates/image/upload.inc?r1=1.5&r2=1.6&ty=u

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/img/upload_preview.php?r1=1.2&r2=1.3&ty=u

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/templates/image/upload.inc?r1=1.4&r2=1.5&ty=u

Once again, I forgot the ticket number in the commit message....if I 
had a dime...

http://lists.horde.org/archives/cvs/Week-of-Mon-20081020/084698.html

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/img/upload_preview.php?r1=1.1&r2=1.2&ty=u
http://cvs.horde.org/diff.php/ansel/templates/image/upload.inc?r1=1.2&r2=1.3&ty=u

I am adding patch to use mini icons and slide show images.

Reduces the height of 240px to 50px and works with custom backgrounds.

For navigation images, what's wrong with the ones in the current image view?



For placement, it needs to be outside the form somewhat so that it's 
not hidden when you switch tabs. The form doesn't need to be 
full-width, though; how about having the image preview about 50% 
width, kind of like the sidebar widgets in the rest of Ansel, have it 
use mini thumbs to save some space, and have the thumbs link to the 
real image in a lightbox?

Since this is merged, this either needs to be enhanced or removed for 1.0

I committed this, and I like the functionality, but I really do not 
like the way it looks.

1) It needs new navigation images since they don't work well with any 
background other than white.

2) I don't like it hanging out below the actual form. I want to add it 
as an html form field, but the size is currently too large for that - 
it's very distracting.



If we can get smaller navigation images, tweak the height of the 
container and maybe use mini images instead of thumbs, I think this 
would be perfect.



Of course, I'm open to any one else's ideas/suggestions...

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ansel/img/upload.php?r1=1.29&r2=1.30&ty=u
http://cvs.horde.org/co.php/ansel/img/upload_preview.php?r=1.1
http://cvs.horde.org/co.php/ansel/js/carousel.js?r=1.1
http://cvs.horde.org/co.php/ansel/js/src/carousel.js?r=1.1
http://cvs.horde.org/co.php/ansel/templates/image/upload.inc?r=1.1
http://cvs.horde.org/co.php/ansel/themes/graphics/carousel/left.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/graphics/carousel/left_disabled.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/graphics/carousel/left_over.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/graphics/carousel/right.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/graphics/carousel/right_disabled.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/graphics/carousel/right_over.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/silver/graphics/carousel/left.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/silver/graphics/carousel/left_disabled.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/silver/graphics/carousel/left_over.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/silver/graphics/carousel/right.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/silver/graphics/carousel/right_disabled.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/silver/graphics/carousel/right_over.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/tango-blue/graphics/carousel/left.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/tango-blue/graphics/carousel/left_disabled.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/tango-blue/graphics/carousel/left_over.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/tango-blue/graphics/carousel/right.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/tango-blue/graphics/carousel/right_disabled.png?r=1.1
http://cvs.horde.org/co.php/ansel/themes/tango-blue/graphics/carousel/right_over.png?r=1.1

Use Ajax calls to load just some images at once.

Some sites has help disabled. Especially no-English pages where help 
translation is often missing.

At least leave the output into a separate page to allow easier customization.

It's better, but this still isn't using *any* dynamic image loading. 
It's outputting *all* of the gallery's image tags at once. For 
galleries with large number of images this could take quite a while, 
especially if the thumbnails still need to be generated.



I also don't agree with putting the help text for XP Publish directly 
on the page. That's what the help system is for.



I'll try and take a look at replacing the carsousel you used with the 
ajax version if I get the time.



Thanks.

Updated to use Prototype-UI. Ajax. Carousel.



I also moved the output to a template file to give more control over 
the content... for example I added the description how to use XPpublish.

seems good.

http://docs.prototype-ui.com/rc0/Carousel/AjaxCarousel



I don't see any reason why we can't use this...

super. Thanks for the CSS.



I'll try to dig around for another alternative for the carousel and 
see what I come up with...otherwise, I'll add this one.



Thanks again!

See attachment.

yes. I was looking for something like that too. But I was unable to 
find a simple prototype script for this. Displaying the last 5 images 
by default would be enough.

I like the idea, but a few issues/questions:



You didn't include the CSS rules you used for the carousel, so I used 
the example from the author's web page, changed the class names to 
match what you used in the code and took some guesses...so, I'm not 
100% sure how you intended this to look on the upload page. Hopefully, 
you have a patch for the css you used?  I'm not crazy about the text 
navigation links, but that's trivial to change.



My real concern is that it took a _very_ long time for the pager to 
load, I suppose because it's loading _all_ the gallery's thumbnails, 
in the gallery's style,  onto the page.  This is not good...If we're 
going to do this, I would prefer some Ajax magic to load the images in 
groups (maybe the same number as defined in the prefs).  To compound 
the  problem, if you have just uploaded large numbers of images, the 
thumbnails are not yet generated, so we are not only loading all the 
thumbnails, we are possibly generating all of them as well.  At the 
very least, if we don't do the ajax calls, we should be forcing the 
plain thumbnails, or better yet, the mini thumbs...anything but the 
'pretty' thumbnails.



I'm wondering if this would be better as a horizontal scrolling 
carousel, with more than one image being visible at a time...something 
that the JS code your using doesn't seem capable of?

Should also be useful for Bug: 4579

When uploading new images, show already loaded images with carousel. 
Very usefull for large galleries when you forgot the last uploaded 
image.

The prototype js plug-in can be dound   
http://www.victorstanciu.ro/prototype-plugin-carousel/