revert r25054 - tags which have an off and a list of option should have off as the first not last option. also fix it so the no icon position actually works.

you can use %?Li<no|yes> if you want to.


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

1 files changed, 2 insertions, 1 deletions

diff --git a/apps/gui/statusbar-skinned.c b/apps/gui/statusbar-skinned.c
index 840ad63c40..0902950599 100644
--- a/apps/gui/statusbar-skinned.c
+++ b/apps/gui/statusbar-skinned.c
@@ -58,7 +58,8 @@ bool sb_set_title_text(char* title, enum themable_icons icon, enum screen_type s
         }
         else if (sb_skin_data[screen].tokens[i].type == WPS_TOKEN_LIST_TITLE_ICON)
         {
-            sb_skin_data[screen].tokens[i].value.i = icon+1;
+            /* Icon_NOICON == -1 which the skin engine wants at position 1, so + 2 */
+            sb_skin_data[screen].tokens[i].value.i = icon+2;
         }
     }
     return retval;