Bugzilla – Bug 13136
Support type=slideshow from SlimBrowser -> Image Viewer
Last modified: 2010-04-08 17:24:13 UTC
We need to define how type=slideshow works.
== Auto-comment from SVN commit #28165 to the slim repo by tom == == https://svn.slimdevices.com/slim?view=revision&revision=28165 == Bug:13136 Description: - (from Andy) If we have a slideshow param, return all items without chunking, and only include image and caption data
== Auto-comment from SVN commit #7033 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7033 == Bug:13136 Description: - Support type=slideshow
== Auto-comment from SVN commit #7038 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7038 == Bug:13136 Description: - revert wrong Image Viewer menu item weight
Remaining work that I think I can do: - preloader: while the slide show is loading, we would use a black screen with the string "loading..." at the bottom. In cases where we have a usable logo for the source (for now this only applies to facebook, flickr, and possibly Queen if Noah can come up with something quick), then we display that asset in the middle of the screen. (see preloader.png). - transitions: by default, we show only a "fade in/fade out" effect. Other transition types could be selected by the user in settings. - after much hemming and hawwing, we've determined it's best for 7.4 to show NO metadata or caption info over the image (there are too many variables from image to image in terms of what's available to display). This info would instead be accessible via a context menu. Available data can include: "[image title]," "Submitted by [user]," "[caption]." **** Question for Matt: How would this context menu be styled (is it just menus items that dont' go anywhere, textareas?) **** - Tom has proposed launching without most of the user-configurable settings for initial launch, other than, I think, delay time. I think it's best to get the basics right first, then add new features later, so this is fine by me. - finally, we'd use the knob/up-down to toggle between photos, and the "back" button to exit the slide show. Not sure about center button/right. I could see this perhaps duplicating the "+" button functionality and launching the context menu...
> **** > Question for Matt: How would this context menu be styled (is it just menus > items that dont' go anywhere, textareas?) > **** Textareas would be best (that's what they'd need to be eventually), but I can live with non-selectable menu items (truncated items would scroll when selected) in the short term.
== Auto-comment from SVN commit #7287 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7287 == Bug:13136 +1 Description: - fade by default - show "Loading..."
== Auto-comment from SVN commit #7307 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7307 == Bug:13136 +2 Description: - show app icon on loading page - added appParameters concept
todo: 1) context menu choice 2) minimal settings
== Auto-comment from SVN commit #7340 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7340 == Bug:13136 +1 Description: - Use minimal settings page for jive,baby
== Auto-comment from SVN commit #7348 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7348 == Bug:13136 +1 Description: - Use minimal settings page for jive,baby (missed commit)
== Auto-comment from SVN commit #7321 to the network repo by andy == == https://svn.slimdevices.com/network?view=revision&revision=7321 == Bug 13136, send discrete date and owner metadata
== Auto-comment from SVN commit #28500 to the slim repo by andy == == https://svn.slimdevices.com/slim?view=revision&revision=28500 == Bug 13136, send discrete date and owner metadata for slideshows
== Auto-comment from SVN commit #7529 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7529 == Bug: 13136 - support context menu for text - default to no text overlay (cm always works though) - SS had bug cleaning up when SS had more than one active window - support date, owner data
Is ready for review
== Auto-comment from SVN commit #7530 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7530 == Bug: 13136 - allow motion in slideshow mode
(In reply to comment #14) > Is ready for review - preloader image is incorrect. on Flickr, Queen and facebook, I see an old app gallery icon that we shouldn't offically be using. We should be using either NO image here, or (in the case of flickr/facebook/queen) use the provided preloader image (Noah has checked these in I think?) - context menus look good. - animations/transitions look good!
The stock image should only appear until the app icon has been cached which in most cases would already have happened. The default image I'm using is the Image Viewer icon, which is a flower in a polaroid. I know of no other default loading image. Noah, do you something else for a default Slideshow/Screensaver loading image?
Tom, The short answer is no. The default would be just the "loading..." text. How feasible would it be to include something for image viewer and Queen if I can get you the assets today or tomorrow? I would prefer not to reuse the App icons in these instances (as seen in the current builds). Engineering ?: I have been embedding the images into the black bkground (Facebook and Flickr). Do you need the assets broken out into separate icons?
Created attachment 5879 [details] Ref artwork - image viewer Not finalized. Proposal of what the default loader could be.
Created attachment 5880 [details] Ref artwork - image viewer (Queen) Not finalized. Will discuss with Weldon today in a separate scheduled meeting.
CC'ing Weldon for comment.
Don't show any image for now, figure out how to fix for 7.4.1
pulling back to 7.4.0 for the remove icon work....
== Auto-comment from SVN commit #7728 to the jive repo by tom == == https://svn.slimdevices.com/jive?view=revision&revision=7728 == Bug: 13136 Description: - don't show any loading icon for 7.4.0
r7728 was reviewed by Ben, btw. Moving to 7.4.1 to restore loading image.
Can someone please describe this problem in plain English, please. Not sure what the bug is here exactly :-) Assigning to Andy since he filed it, but any comments are welcome...
This is still open to re-add the loading icons, see above.
Loading icons (what's left with this bug) doesn't seem like it merits this as a P1
== Auto-comment from SVN commit #8485 to the repo by mherger == == https://svn.slimdevices.com/?view=revision&revision=8485 == Fixed Bug: 13136 Description: add loading icon to ImageViewer. Use app specific icon if possible.
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release. If you are still seeing this issue, please let us know!