Bug 11164 - Remove all misc. wallpapers
: Remove all misc. wallpapers
Status: RESOLVED FIXED
Product: SB Touch
Classification: Unclassified
Component: UI
: unspecified
: All Other
: -- normal (vote)
: 8.0.0
Assigned To: Richard Titmuss
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-02-23 14:03 UTC by ndijulio
Modified: 2009-09-08 09:13 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-02-23 14:03:23 UTC
The only wallpapers that should be in the current UI build are:

1. Sunrise

2. Wave Pattern 1
Comment 1 Ben Klaas 2009-02-23 14:20:55 UTC
FYI, the wallpaper is served up by SetupWallpaper, which is not a Fab4-specific applet. On this we'll have to pay attention to proper support for Fab4 as well as controller (and desktop, and future products)
Comment 2 Blackketter Dean 2009-02-23 14:32:00 UTC
I think there are two issues here:

1.  Handling platform specific rendering, which has more to do with screen size than anything else and dealing with stretching vs tiling, etc.  Code.

2.  Building device-specific firmware images that include the appropriate wallpaper graphics resources for the given target.   Build environment.
Comment 3 Richard Titmuss 2009-03-26 04:01:46 UTC
I'll look at the platform build issue. We should also remove move wallpapers from the firmware, with all the additional wallpapers install from SC/SN.
Comment 4 Richard Titmuss 2009-03-26 04:36:20 UTC
*** Bug 11204 has been marked as a duplicate of this bug. ***
Comment 5 Ben Klaas 2009-05-23 05:50:29 UTC
This is fixed in r5855. Gory details below.


Author: bklaas
Date: Fri May 22 21:32:56 2009
New Revision: 5855

Log:
Bug: 11164
Description: achieve platform-specific wallpaper sets by creating share/applets/SetupWallpaper/wallpaper directory in
platform specific source areas and leaving applet code itself in squeezeplay/ area. SetupWallpaperApplet.lua removes hard-coded wallpaper table and replaces it with a dynamic reading of the wallpaper/ directory in the applet path.

solid color wallpaper and images that tile well are still in squeezeplay/
240x320 wallpapers are in squeezeplay_jive/
480x272 wallpapers are in squeezeplay_fab4/
jive and fab4 wallpapers are both currently copied to squeezeplay_desktop/ so they are all available on the desktop, which can choose any of these skins

strings.txt file in squeezeplay/ is "monolithic", in that it contains string translations for all wallpaper titles, no matter what the platform. Token naming convention is PLATFORM_WALLPAPERNAME, which must map directly to wallpapers/platform_wallpapername.(gif|jpg|png)

import new fab4-specific wallpapers from Noah
Comment 6 Blackketter Dean 2009-05-23 09:02:45 UTC
Any way to make the strings file not monolithic?
Comment 7 Blackketter Dean 2009-05-23 09:05:24 UTC
To be clearer: I think that we should be able to add a new group of wallpapers to one or more platforms and have the strings tied to the wallpapers, not necessarily centrally managed.

This implies to me that we need to be able to load more than one strings.txt file per applet (i.e. treat strings like data and not like code)
Comment 8 ndijulio 2009-05-27 09:48:12 UTC
Is this still in the works?  There are a number of wallpapers on fab4 that are carry overs from Controller.  Please keep in mind that Fab4 was not designed to support these wallpapers and these should be removed.
Comment 9 Ben Klaas 2009-05-27 09:49:02 UTC
Noah, which wallpapers?

IMO, this is feature complete.
Comment 10 ndijulio 2009-05-27 09:55:45 UTC
If it is not:

1. wallpaper_allegro
2. wallpaper_encore
3. wallpaper_harmony
4. wallpaper_nocturne (DEFAULT)
5. wallpaper_quartet
6. wallpaper_sonata
7. wallpaper_symphony
8. wallpaper_vibrato

It should be removed.  My fab4 unit is locked or I would call-out the specific images...
Comment 11 Ben Klaas 2009-05-27 09:58:43 UTC
so not even solid colors?

I'd left all images that tiled well as platform-inspecific. Those aren't acceptable options for Fab4?
Comment 12 ndijulio 2009-05-27 10:17:48 UTC
Yes, even the solid colors.  We can add these back to mysqueezebox.com, however, they should not be included in the out-of-box.
Comment 13 Ben Klaas 2009-05-27 10:19:11 UTC
disagree, but fine. I'll make the change.
Comment 14 Ben Klaas 2009-05-27 13:50:02 UTC
done, r5910
Comment 15 Blackketter Dean 2009-05-27 15:30:15 UTC
Thanks, Ben.  Let's get back to basics for Jive, Fab4 and Baby and then
review what we have and see where we need to make additions.

On Wed, May 27, 2009 at 1:50 PM, <bugs@bugs.slimdevices.com> wrote:

> https://bugs-archive.lyrion.org/show_bug.cgi?id=11164
>
>
> Ben Klaas <bklaas@slimdevices.com> changed:
>
>           What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>              Status|REOPENED                    |RESOLVED
>         Resolution|                            |FIXED
>
>
>
>
> --- Comment #14 from Ben Klaas <bklaas@slimdevices.com>  2009-05-27
> 13:50:02 ---
> done, r5910
>
> --
> Configure bugmail: https://bugs-archive.lyrion.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
>