len0rd/rockbox
1
0
Fork 1
mirror of https://github.com/Rockbox/rockbox.git synced 2025-12-06 21:25:03 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

touchscreen: Default to putting the scrollbar on the right

Browse source 
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
This commit is contained in:
Aidan MacDonald 2022-04-20 10:50:07 +01:00 • committed by Solomon Peachy
parent 9290a484e4
commit 9f613fd981
1 changed files with 8 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

9
apps/settings_list.c
View file

@ -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,