creativepackfz / risegadgets

Automatically exported from code.google.com/p/risegadgets
0 stars 0 forks source link

Photo album images do not show in order of the feed #718

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Open the feed in Firefox and notice the order images get displayed, by 
filename. 
https://picasaweb.google.com/data/feed/base/user/115516543632082636933/albumid/5
758023209814557905?alt=rss&kind=photo&hl=en_US&imgmax=d
2. Create a new presentation and use the same feed with two gadgets, Photo 
Album Gadget and RSS Gadget.
3. Observe the difference in the order images get presented. Test presentation 
here: 
http://rdn-test.appspot.com/#PRESENTATION_MANAGE/id=63b9d3f7-15c8-4910-8ece-ef6e
ec321d1b/company=fb788f1f-7730-44fd-8e00-20066409f51f

What is the expected output? What do you see instead?
--The RSS Gadget displays images in the order of the feed, while the Photo 
Album displays them in seemingly random order.

What version of the product are you using? On what operating system?
--Rise Vision Photo Album - TEST in RVA Testing shared gadgets. Ubuntu 12.04.

Please provide any additional information below.
--We tried to manipulate the order images get sorted in the feed by appending 
"&orderby=date". Some Picasa users reported that adding this parameter to 
Picasa rss feed would make images sort by date, as in this example:
http://picasaweb.google.com/data/feed/api/all?q=paris&max-results=5
http://picasaweb.google.com/data/feed/api/all?q=paris&max-results=5&orderby=date
However, in our example images are already sorted the way the user wanted them 
sorted, by filename. And the orderby option didn't work for their feed anyway.

Original issue reported on code.google.com by mark.kre...@risevision.com on 23 Aug 2012 at 1:41

GoogleCodeExporter commented 9 years ago

Original comment by robb.pr...@risevision.com on 23 Aug 2012 at 1:45

GoogleCodeExporter commented 9 years ago
Yes, I recall at the time that this Gadget was first created, we knew this 
problem with the order. The reason for this is that all images are loaded 
asynchronously. The order that they load in is the order that they are 
displayed in.

Original comment by donnapep@gmail.com on 31 Aug 2012 at 3:31

GoogleCodeExporter commented 9 years ago
Fixed for slideshow and collage. Cover flow seemed fine.
Also, fixed another issue with the collage speeding up after the first cycle.

Original comment by donnapep@gmail.com on 5 Sep 2012 at 1:49

GoogleCodeExporter commented 9 years ago
The gadget seems to slice some of the pictures, instead of displaying them in 
full size or scaling them down. Also, a lot of times the pictures get skipped. 
Those should play on the 2nd run through the feed, if they failed to download 
on the first.

http://screencast.com/t/YSzAv1S1V

And here's the link to the presentation used for testing:
http://rdn-test.appspot.com/#PRESENTATION_MANAGE/id=63b9d3f7-15c8-4910-8ece-ef6e
ec321d1b/company=fb788f1f-7730-44fd-8e00-20066409f51f

Original comment by mark.kre...@risevision.com on 5 Sep 2012 at 9:38

GoogleCodeExporter commented 9 years ago
http://screencast.com/t/WiTaZLMaAyNn

Original comment by mark.kre...@risevision.com on 5 Sep 2012 at 10:09

GoogleCodeExporter commented 9 years ago
Items are not being skipped. It just looks like they're being skipped because 
they're mostly white and not centered.

Original comment by donnapep@gmail.com on 6 Sep 2012 at 1:37

GoogleCodeExporter commented 9 years ago

Original comment by donnapep@gmail.com on 6 Sep 2012 at 1:37

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Here’s the dilemma with this one. We currently show the Gadget after 10 
images have loaded, any 10, so that the Gadget isn’t blank while waiting for 
all images to load. Now, if we want to show them in the order that they appear 
in the feed, we have to wait for all of them to load. So there’s a tradeoff 
here - show images in the order of the feed, or load the Gadget quickly. Which 
do we want to compromise?

Original comment by donnapep@gmail.com on 6 Sep 2012 at 2:18

GoogleCodeExporter commented 9 years ago
I tested with a feed that the customer reported as not being played in order:
https://picasaweb.google.com/data/feed/base/user/115516543632082636933/albumid/5
758023209814557905?alt=rss&kind=photo&hl=en_US&imgmax=d

The white light-weighted images would load just fine in my testing. It's the 
portraits that would get sliced. BTW, the existing gadget would play all images 
in this same feed properly, without slicing them, just not in order of the feed.

To try and resolve the dilemma we could start playing the first 10 images that 
load. But then the expectation is that once all images get loaded (after the 
first run thru the loop?), they all should start playing in order of the feed. 
So, it looks like the gadget should keep the old behavior upon the first run, 
to start playing something quickly, but also to be smart enough to figure out 
the right order, once all images get downloaded into cache.

Original comment by mark.kre...@risevision.com on 6 Sep 2012 at 3:07

GoogleCodeExporter commented 9 years ago
Changed it so that, if there are 10+ photos in the feed, the 10 that load 
fastest show first. On subsequent cycles, the images show in the same order as 
the feed. Note though, that this introduces a longer delay after all of the 
photos are shown, since the images are not loaded by the browser in the same 
order that they are requested.

Original comment by donnapep@gmail.com on 10 Sep 2012 at 6:01

GoogleCodeExporter commented 9 years ago
Realized I forgot to handle the case with < 10 images.

Original comment by donnapep@gmail.com on 10 Sep 2012 at 8:26

GoogleCodeExporter commented 9 years ago
My bad. Same code covers all cases.

Original comment by donnapep@gmail.com on 10 Sep 2012 at 8:31

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago

Original comment by neal.god...@risevision.com on 11 Sep 2012 at 2:15

GoogleCodeExporter commented 9 years ago

Original comment by donnapep@gmail.com on 11 Sep 2012 at 11:31