revert r24478 and make the backdrop behaviour the same as it used to be (so when in the wps the sbs' backdrop will be fully disabled). Also changes the API back to having to manually set the current backdrop.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@24565 a1c6a512-1295-4272-9138-f99709370657

4 files changed, 6 insertions, 9 deletions

diff --git a/apps/gui/skin_engine/skin_display.c b/apps/gui/skin_engine/skin_display.c
index 9ec4a9e..8637190 100644
--- a/apps/gui/skin_engine/skin_display.c
+++ b/apps/gui/skin_engine/skin_display.c
@@ -1006,11 +1006,6 @@ static bool skin_redraw(struct gui_wps *gwps, unsigned refresh_mode)
 
 #endif
 
-#if (LCD_DEPTH > 1) || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)
-    display->backdrop_show(data->backdrop);
-#endif
-
-
     /* reset to first subline if refresh all flag is set */
     if (refresh_mode == WPS_REFRESH_ALL)
     {
diff --git a/apps/gui/viewport.c b/apps/gui/viewport.c
index 78b6379..4b89608 100644
--- a/apps/gui/viewport.c
+++ b/apps/gui/viewport.c
@@ -111,9 +111,6 @@ static void toggle_theme(enum screen_type screen, bool force)
     {
         /* remove the left overs from the previous screen.
          * could cause a tiny flicker. Redo your screen code if that happens */
-#if (LCD_DEPTH > 1) || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)
-        screens[screen].backdrop_show(sb_get_backdrop(screen));
-#endif
         if (!was_enabled[screen] || force)
         {
             struct viewport deadspace, user;
diff --git a/apps/gui/wps.c b/apps/gui/wps.c
index ca74c45..f461c02 100644
--- a/apps/gui/wps.c
+++ b/apps/gui/wps.c
@@ -565,6 +565,7 @@ static void gwps_leave_wps(void)
     FOR_NB_SCREENS(i)
     {
         gui_wps[i].display->stop_scroll();
+        gui_wps[i].display->backdrop_show(sb_get_backdrop(i));
         
 #ifdef HAVE_LCD_BITMAP
         bool draw = false;
@@ -601,9 +602,10 @@ static void gwps_enter_wps(void)
         else if (statusbar_position(i) != STATUSBAR_OFF)
             draw = true;
 #endif
+        display->stop_scroll();
+        display->backdrop_show(gwps->data->backdrop);
         viewportmanager_theme_enable(i, draw, NULL);
 
-        display->stop_scroll();
         /* Update the values in the first (default) viewport - in case the user
            has modified the statusbar or colour settings */
 #if LCD_DEPTH > 1
diff --git a/apps/settings.c b/apps/settings.c
index 1b16e9f..c4ebabc 100644
--- a/apps/settings.c
+++ b/apps/settings.c
@@ -737,6 +737,7 @@ void sound_settings_apply(void)
 void settings_apply_skins(void)
 {
     char buf[MAX_PATH];
+    int i;
     /* re-initialize the skin buffer before we start reloading skins */
     skin_buffer_init();
 #ifdef HAVE_LCD_BITMAP
@@ -786,6 +787,8 @@ void settings_apply_skins(void)
     }
 #endif
     viewportmanager_theme_changed(THEME_STATUSBAR);
+    FOR_NB_SCREENS(i)
+        screens[i].backdrop_show(sb_get_backdrop(i));
 }
 
 void settings_apply(bool read_disk)