Touchscreen list handling: use icon width instead of scrollbar width for the 'ignoring' zone + correct comment that shouldn't be true anymore (and never was AFAIK)

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

1 files changed, 2 insertions, 5 deletions

diff --git a/apps/gui/bitmap/list.c b/apps/gui/bitmap/list.c
index 2a3a31a548..29b97a8fa3 100644
--- a/apps/gui/bitmap/list.c
+++ b/apps/gui/bitmap/list.c
@@ -285,10 +285,7 @@ void list_draw(struct screen *display, struct gui_synclist *list)
 }
 
 #if defined(HAVE_TOUCHSCREEN)
-/* This needs to be fixed if we ever get more than 1 touchscreen on a target.
- * This also assumes the whole screen is used, which is a bad assumption but
- * fine until customizable lists comes in...
- */
+/* This needs to be fixed if we ever get more than 1 touchscreen on a target. */
 static bool scrolling=false;
 
 static int gui_synclist_touchscreen_scrollbar(struct gui_synclist * gui_list,
@@ -400,7 +397,7 @@ unsigned gui_synclist_do_touchscreen(struct gui_synclist * gui_list)
              */
             if(global_settings.scrollbar == SCROLLBAR_RIGHT &&
                x > list_text[screen].x + list_text[screen].width -
-                   global_settings.scrollbar_width)
+                   get_icon_width(SCREEN_MAIN))
                 return ACTION_NONE;
             
             if (button == (BUTTON_REPEAT|BUTTON_REL))