From 9f613fd981b4c3c32c8fb77040d8878ef85ca85d Mon Sep 17 00:00:00 2001 From: Aidan MacDonald Date: Wed, 20 Apr 2022 10:50:07 +0100 Subject: [PATCH] touchscreen: Default to putting the scrollbar on the right For touch targets and right-handed use it makes most sense to put the scrollbar on the right so your finger doesn't get in the way of the touchscreen. Sorry left-handed people! Change-Id: I35cfd8aa3bee6eb638572c9ca592942c53188a50 --- apps/settings_list.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/apps/settings_list.c b/apps/settings_list.c index e74c70c448..a029cfa7fa 100644 --- a/apps/settings_list.c +++ b/apps/settings_list.c @@ -480,6 +480,13 @@ static const char graphic_numeric[] = "graphic,numeric"; # define MAX_FILES_IN_DIR_STEP 50 #endif +#ifdef HAVE_TOUCHSCREEN +/* on touchscreen, it makes more sense to put the scrollbar on the right */ +# define SCROLLBAR_DEFAULT SCROLLBAR_RIGHT +#else +# define SCROLLBAR_DEFAULT SCROLLBAR_LEFT +#endif + #ifndef __PCTOOL__ #if LCD_DEPTH > 1 @@ -1190,7 +1197,7 @@ const struct settings_list settings[] = { ID2P(LANG_STATUSBAR_BOTTOM)), #endif CHOICE_SETTING(F_THEMESETTING|F_TEMPVAR, scrollbar, - LANG_SCROLL_BAR, SCROLLBAR_LEFT, "scrollbar","off,left,right", + LANG_SCROLL_BAR, SCROLLBAR_DEFAULT, "scrollbar","off,left,right", NULL, 3, ID2P(LANG_OFF), ID2P(LANG_LEFT), ID2P(LANG_RIGHT)), INT_SETTING(F_THEMESETTING, scrollbar_width, LANG_SCROLLBAR_WIDTH, 6, "scrollbar width",UNIT_INT, 3, MAX(LCD_WIDTH/10,25), 1,