6.0.0-beta1
7/12/25

[#7967] Upload UI
Summary Upload UI
Queue Ansel
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester duck (at) obala (dot) net
Created 02/09/2009 (5997 days ago)
Due
Updated 03/09/2009 (5969 days ago)
Assigned
Resolved 03/09/2009 (5969 days ago)
Milestone
Patch Yes

History
03/09/2009 02:29:47 PM Chuck Hagenbuch Comment #10
State ⇒ Resolved
Reply to this comment
Closing for now.
03/09/2009 02:28:41 PM Michael Rubinsky Comment #9 Reply to this comment
I'm inclined to say this ticket can be closed.  Having a lightbox on 
the page would be nice, but would pretty much require a total rewrite 
of the ajax carousel, not something I'm personally up for in the near 
future.



First/Last links would also be nice, but again, would require some 
changes to the carousel code that I'm not inclined to spend time on in 
the near future.



So, unless anyone else is going to do this, I'd say this can be 
closed, or maybe given a 2.0 milestone...
03/09/2009 12:45:02 AM Chuck Hagenbuch Comment #8 Reply to this comment
What's left here?
02/10/2009 02:58:50 PM Michael Rubinsky Comment #7 Reply to this comment
If ease of navigation here is the goal - then we should probably add a 
first and last image link to the carousel...
02/10/2009 02:56:17 PM Michael Rubinsky Comment #6 Reply to this comment
I guess I don't know if I'm comfortable with changing the user-defined 
sort order...plus the image_id will only follow the larger the numer 
== more recent upload if there were no images moved into the gallery 
from another gallery - probably a somewhat fringe case, I know, but I 
don't think we should be relying on a backend specific numbering 
scheme for presentation logic.




02/10/2009 10:34:25 AM Duck Comment #5 Reply to this comment
The only issue I have with this is that the images are not
necessarily listed in chronological order, they are listed according
to the image_sort field.  Listing them in reverse order may or may
not provide the last uploaded image first - and it might lead to
confusion with the user as to why there images are not in the order
they have selected.
Then we can just resort the array of images_ids. As ids are 
incremental, we get the correct upload order by using 
rsort($image_ids, SORT_NUMERIC).


02/10/2009 01:39:35 AM Michael Rubinsky Comment #4 Reply to this comment
Committed item #2 from above, and tweaked it a bit to remove some 
(now) redundant checks in the upload template file and to avoid 
loading any of the js if we aren't showing the carousel.



Anyone else want to weigh in on points #1 and #3 before I close?
02/09/2009 05:18:06 PM Michael Rubinsky Assigned to Michael Rubinsky
State ⇒ Feedback
 
02/09/2009 05:16:59 PM Michael Rubinsky Comment #2 Reply to this comment
1.redirect again to image upload screen after image upload. Is a
natural begaviour as you are filling the gallery.
I'm not sure I agree with this.  If we are going to change this 
behavior, I'd much rather see something like a "What do you want to do 
now?" message. You could redirect back to the upload screen, and 
display this message with choices like view gallery, add captions, 
etc...

This is how Flickr works, IIRC.



Of course, even better would be an Ajax image uploader :)
2.Load image_ids for lightbox only if gallery is  not empty
Looks good, thanks.
3.Show images in recursive order. Is more natural to see by default
the last uploaded images not to always slide to see what you uploaded
the last time.
The only issue I have with this is that the images are not necessarily 
listed in chronological order, they are listed according to the 
image_sort field.  Listing them in reverse order may or may not 
provide the last uploaded image first - and it might lead to confusion 
with the user as to why there images are not in the order they have 
selected.
02/09/2009 10:33:36 AM Duck Comment #1
Priority ⇒ 1. Low
State ⇒ New
New Attachment: UPLOAD.diff Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Ansel
Summary ⇒ Upload UI
Type ⇒ Enhancement
Reply to this comment
1.redirect again to image upload screen after image upload. Is a 
natural begaviour as you are filling the gallery. Otherwise you must 
always go back to the upload screen.

2.Load image_ids for lightbox only if gallery is  not empty

3.Show images in recursive order. Is more natural to see by default 
the last uploaded images not to always slide to see what you uploaded 
the last time.

Saved Queries