From f5ec3e49e0687eda7b9059942c23321f9878becf Mon Sep 17 00:00:00 2001 From: Jonathan Gordon Date: Mon, 17 Dec 2007 02:37:21 +0000 Subject: [PATCH] remove the duplicated gui_list struct and only duplicate the members which are actually different on each screen. should translate to a smaller bin and easier code (users shouldnt notice any difference) git-svn-id: svn://svn.rockbox.org/rockbox/trunk@15947 a1c6a512-1295-4272-9138-f99709370657 --- apps/debug_menu.c | 8 +- apps/filetypes.c | 2 +- apps/gui/list.c | 509 ++++++++++++++++------------------------- apps/gui/list.h | 22 +- apps/menus/main_menu.c | 2 +- apps/tree.c | 2 +- 6 files changed, 206 insertions(+), 339 deletions(-) diff --git a/apps/debug_menu.c b/apps/debug_menu.c index a8eb786f83..8f24ac95f6 100644 --- a/apps/debug_menu.c +++ b/apps/debug_menu.c @@ -1449,11 +1449,11 @@ static char* tsc2100_debug_getname(int selected_item, void * data, char *buffer) } static int tsc2100debug_action_callback(int action, struct gui_synclist *lists) { - int *page = (int*)lists->gui_list[SCREEN_MAIN].data; + int *page = (int*)lists->data; if (action == ACTION_STD_OK) { *page = (*page+1)%3; - snprintf(lists->gui_list[SCREEN_MAIN].title, 32, + snprintf(lists->title, 32, "tsc2100 registers - Page %d", *page); return ACTION_REDRAW; } @@ -1667,10 +1667,10 @@ static bool view_battery(void) static int disk_callback(int btn, struct gui_synclist *lists) { tCardInfo *card; - int *cardnum = (int*)lists->gui_list[SCREEN_MAIN].data; + int *cardnum = (int*)lists->data; unsigned char card_name[7]; unsigned char pbuf[32]; - char *title = lists->gui_list[SCREEN_MAIN].title; + char *title = lists->title; static const unsigned char i_vmin[] = { 0, 1, 5, 10, 25, 35, 60, 100 }; static const unsigned char i_vmax[] = { 1, 5, 10, 25, 35, 45, 80, 200 }; static const unsigned char *kbit_units[] = { "kBit/s", "MBit/s", "GBit/s" }; diff --git a/apps/filetypes.c b/apps/filetypes.c index 5c74103cf1..db463d7ff4 100644 --- a/apps/filetypes.c +++ b/apps/filetypes.c @@ -443,7 +443,7 @@ char * openwith_get_name(int selected_item, void * data, char * buffer) } int openwith_action_callback(int action, struct gui_synclist *lists) { - struct cb_data *info = (struct cb_data *)lists->gui_list[SCREEN_MAIN].data; + struct cb_data *info = (struct cb_data *)lists->data; int *items = info->items; int i; if (action == ACTION_STD_OK) diff --git a/apps/gui/list.c b/apps/gui/list.c index 86ef97cf4d..6fdc7a10a5 100644 --- a/apps/gui/list.c +++ b/apps/gui/list.c @@ -54,12 +54,13 @@ static int offset_step = 16; /* pixels per screen scroll step */ /* should lines scroll out of the screen */ static bool offset_out_of_view = false; #endif -static struct gui_list* last_list_displayed[NB_SCREENS]; +static struct gui_synclist* last_list_displayed; #define SHOW_LIST_TITLE ((gui_list->title != NULL) && \ - (gui_list->display->nb_lines > 2)) + (display->nb_lines > 2)) -static void gui_list_select_at_offset(struct gui_list * gui_list, int offset); +static void gui_list_select_at_offset(struct gui_synclist * gui_list, + int offset); /* * Initializes a scrolling list @@ -70,35 +71,38 @@ static void gui_list_select_at_offset(struct gui_list * gui_list, int offset); * - scroll_all : * - selected_size : */ -static void gui_list_init(struct gui_list * gui_list, +void gui_synclist_init(struct gui_synclist * gui_list, list_get_name callback_get_item_name, void * data, bool scroll_all, int selected_size ) { + int i; gui_list->callback_get_item_icon = NULL; gui_list->callback_get_item_name = callback_get_item_name; gui_list->callback_speak_item = NULL; - gui_list->display = NULL; gui_list_set_nb_items(gui_list, 0); gui_list->selected_item = 0; - gui_list->start_item = 0; + FOR_NB_SCREENS(i) + { + gui_list->start_item[i] = 0; + gui_list->last_displayed_start_item[i] = -1 ; +#ifdef HAVE_LCD_BITMAP + gui_list->offset_position[i] = 0; +#endif + } gui_list->limit_scroll = false; gui_list->data=data; -#ifdef HAVE_LCD_BITMAP - gui_list->offset_position = 0; -#endif gui_list->scroll_all=scroll_all; gui_list->selected_size=selected_size; gui_list->title = NULL; gui_list->title_width = 0; gui_list->title_icon = Icon_NOICON; - gui_list->last_displayed_selected_item = -1 ; - gui_list->last_displayed_start_item = -1 ; gui_list->scheduled_talk_tick = gui_list->last_talked_tick = 0; gui_list->show_selection_marker = true; + gui_list->last_displayed_selected_item = -1 ; #ifdef HAVE_LCD_COLOR gui_list->title_color = -1; @@ -109,38 +113,19 @@ static void gui_list_init(struct gui_list * gui_list, /* this toggles the selection bar or cursor */ void gui_synclist_hide_selection_marker(struct gui_synclist * lists, bool hide) { - int i; - FOR_NB_SCREENS(i) - lists->gui_list[i].show_selection_marker = !hide; + lists->show_selection_marker = !hide; } -/* - * Attach the scrolling list to a screen - * (The previous screen attachement is lost) - * - gui_list : the list structure - * - display : the screen to attach - */ -static void gui_list_set_display(struct gui_list * gui_list, struct screen * display) -{ - if(gui_list->display != 0) /* we switched from a previous display */ - gui_list->display->stop_scroll(); - gui_list->display = display; -#ifdef HAVE_LCD_CHARCELLS - display->double_height(false); -#endif - gui_list_select_at_offset(gui_list, 0); -} #ifdef HAVE_LCD_BITMAP -static int gui_list_get_item_offset(struct gui_list * gui_list, int item_width, - int text_pos) +static int gui_list_get_item_offset(struct gui_synclist * gui_list, int item_width, + int text_pos, struct screen * display) { - struct screen * display=gui_list->display; int item_offset; if (offset_out_of_view) { - item_offset = gui_list->offset_position; + item_offset = gui_list->offset_position[display->screen_type]; } else { @@ -152,11 +137,11 @@ static int gui_list_get_item_offset(struct gui_list * gui_list, int item_width, else { /* if text got out of view */ - if (gui_list->offset_position > + if (gui_list->offset_position[display->screen_type] > item_width - (display->width - text_pos)) item_offset = item_width - (display->width - text_pos); else - item_offset = gui_list->offset_position; + item_offset = gui_list->offset_position[display->screen_type]; } } @@ -168,9 +153,8 @@ static int gui_list_get_item_offset(struct gui_list * gui_list, int item_width, * Draws the list on the attached screen * - gui_list : the list structure */ -static void gui_list_draw_smart(struct gui_list *gui_list) +static void gui_list_draw_smart(struct gui_synclist *gui_list, struct screen * display) { - struct screen * display=gui_list->display; int text_pos; bool draw_icons = (gui_list->callback_get_item_icon != NULL && global_settings.show_icons); bool draw_cursor; @@ -189,8 +173,8 @@ static void gui_list_draw_smart(struct gui_list *gui_list) gui_textarea_update_nblines(display); #endif /* Speed up UI by drawing the changed contents only. */ - if (gui_list == last_list_displayed[gui_list->display->screen_type] - && gui_list->last_displayed_start_item == gui_list->start_item + if (gui_list == last_list_displayed + && gui_list->last_displayed_start_item[display->screen_type] == gui_list->start_item[display->screen_type] && gui_list->selected_size == 1) { partial_draw = true; @@ -205,8 +189,8 @@ static void gui_list_draw_smart(struct gui_list *gui_list) if (partial_draw) { - end = gui_list->last_displayed_selected_item - gui_list->start_item; - i = gui_list->selected_item - gui_list->start_item; + end = gui_list->last_displayed_selected_item - gui_list->start_item[display->screen_type]; + i = gui_list->selected_item - gui_list->start_item[display->screen_type]; if (i < end ) { start = i; @@ -223,8 +207,8 @@ static void gui_list_draw_smart(struct gui_list *gui_list) gui_textarea_clear(display); start = 0; end = display->nb_lines; - gui_list->last_displayed_start_item = gui_list->start_item; - last_list_displayed[gui_list->display->screen_type] = gui_list; + gui_list->last_displayed_start_item[display->screen_type] = gui_list->start_item[display->screen_type]; + last_list_displayed = gui_list; } gui_list->last_displayed_selected_item = gui_list->selected_item; @@ -257,7 +241,7 @@ static void gui_list_draw_smart(struct gui_list *gui_list) #endif screen_set_xmargin(display, text_pos); /* margin for title */ item_offset = gui_list_get_item_offset(gui_list, gui_list->title_width, - text_pos); + text_pos, display); if (item_offset > gui_list->title_width - (display->width - text_pos)) display->puts_style_offset(0, 0, gui_list->title, title_style, item_offset); @@ -317,7 +301,7 @@ static void gui_list_draw_smart(struct gui_list *gui_list) unsigned char *s; char entry_buffer[MAX_PATH]; unsigned char *entry_name; - int current_item = gui_list->start_item + + int current_item = gui_list->start_item[display->screen_type] + (SHOW_LIST_TITLE ? i-1 : i); /* When there are less items to display than the @@ -334,7 +318,8 @@ static void gui_list_draw_smart(struct gui_list *gui_list) /* position the string at the correct offset place */ int item_width,h; display->getstringsize(entry_name, &item_width, &h); - item_offset = gui_list_get_item_offset(gui_list, item_width, text_pos); + item_offset = gui_list_get_item_offset(gui_list, item_width, + text_pos, display); #endif #ifdef HAVE_LCD_COLOR @@ -466,8 +451,8 @@ static void gui_list_draw_smart(struct gui_list *gui_list) lines + y_start; gui_scrollbar_draw(display, 0, y_start, SCROLLBAR_WIDTH-1, scrollbar_y_end - y_start, gui_list->nb_items, - gui_list->start_item, - gui_list->start_item + lines, VERTICAL); + gui_list->start_item[display->screen_type], + gui_list->start_item[display->screen_type] + lines, VERTICAL); } screen_set_xmargin(display, old_margin); @@ -479,175 +464,149 @@ static void gui_list_draw_smart(struct gui_list *gui_list) /* * Force a full screen update. */ -static void gui_list_draw(struct gui_list *gui_list) +void gui_synclist_draw(struct gui_synclist *gui_list) { - last_list_displayed[gui_list->display->screen_type] = NULL; - return gui_list_draw_smart(gui_list); + int i; + FOR_NB_SCREENS(i) + { + last_list_displayed = NULL; + gui_list_draw_smart(gui_list, &screens[i]); + } } -/* - * Selects an item in the list - * - gui_list : the list structure - * - item_number : the number of the item which will be selected - */ -static void gui_list_select_item(struct gui_list * gui_list, int item_number) -{ - if( item_number > gui_list->nb_items-1 || item_number < 0 ) - return; - gui_list->selected_item = item_number; - gui_list_select_at_offset(gui_list, 0); -} -/* select an item above the current one */ -static void gui_list_select_above(struct gui_list * gui_list, - int items, int nb_lines) + +/* sets up the list so the selection is shown correctly on the screen */ +static void gui_list_put_selection_on_screen(struct gui_synclist * gui_list, + enum screen_type screen) { - gui_list->selected_item -= items; + struct screen *display = &screens[screen]; + int nb_lines = display->nb_lines - SHOW_LIST_TITLE; + int difference = gui_list->selected_item - gui_list->start_item[screen]; - /* in bottom "3rd" of the screen, so dont move the start item. - by 3rd I mean above SCROLL_LIMIT lines above the end of the screen */ - if (items && gui_list->start_item + SCROLL_LIMIT < gui_list->selected_item) + /* edge case,, selected last item */ + if (gui_list->selected_item == gui_list->nb_items -1) { - if (gui_list->show_selection_marker == false) - { - gui_list->start_item -= items; - if (gui_list->start_item < 0) - gui_list->start_item = 0; - } - return; + gui_list->start_item[screen] = MAX(0, gui_list->nb_items - nb_lines); } - if (gui_list->selected_item < 0) + /* selected first item */ + else if (gui_list->selected_item == 0) { - if(gui_list->limit_scroll) - { - gui_list->selected_item = 0; - gui_list->start_item = 0; - } - else - { - gui_list->selected_item += gui_list->nb_items; - if (global_settings.scroll_paginated) - { - gui_list->start_item = gui_list->nb_items - nb_lines; - } - } + gui_list->start_item[screen] = 0; } - if (gui_list->nb_items > nb_lines) + else if (difference < SCROLL_LIMIT) /* list moved up */ { if (global_settings.scroll_paginated) { - if (gui_list->start_item > gui_list->selected_item) - gui_list->start_item = MAX(0, gui_list->start_item - nb_lines); + if (gui_list->start_item[screen] > gui_list->selected_item) + gui_list->start_item[screen] = MAX(0, gui_list->start_item[screen] - nb_lines); } else { int top_of_screen = gui_list->selected_item - SCROLL_LIMIT; int temp = MIN(top_of_screen, gui_list->nb_items - nb_lines); - gui_list->start_item = MAX(0, temp); + gui_list->start_item[screen] = MAX(0, temp); } } - else gui_list->start_item = 0; - if (gui_list->selected_size > 1) - { - if (gui_list->start_item + nb_lines == gui_list->selected_item) - gui_list->start_item++; - } -} -/* select an item below the current one */ -static void gui_list_select_below(struct gui_list * gui_list, - int items, int nb_lines) -{ - int bottom; - - gui_list->selected_item += items; - bottom = gui_list->nb_items - nb_lines; - - /* always move the screen if selection isnt "visible" */ - if (items && gui_list->show_selection_marker == false) - { - if (bottom < 0) - bottom = 0; - gui_list->start_item = MIN(bottom, gui_list->start_item + - items); - return; - } - /* in top "3rd" of the screen, so dont move the start item */ - if (items && - (gui_list->start_item + nb_lines - SCROLL_LIMIT > gui_list->selected_item) - && (gui_list->selected_item < gui_list->nb_items)) + else if (difference > nb_lines - SCROLL_LIMIT) /* list moved down */ { + int bottom = gui_list->nb_items - nb_lines; + /* always move the screen if selection isnt "visible" */ if (gui_list->show_selection_marker == false) { if (bottom < 0) bottom = 0; - gui_list->start_item = MIN(bottom, - gui_list->start_item + items); + gui_list->start_item[screen] = MIN(bottom, gui_list->start_item[screen] + + 2*gui_list->selected_size); } - return; - } - - if (gui_list->selected_item >= gui_list->nb_items) - { - if(gui_list->limit_scroll) + else if (global_settings.scroll_paginated) { - gui_list->selected_item = gui_list->nb_items-gui_list->selected_size; - gui_list->start_item = MAX(0,gui_list->nb_items - nb_lines); - } - else - { - gui_list->selected_item = 0; - gui_list->start_item = 0; - } - return; - } - - if (gui_list->nb_items > nb_lines) - { - if (global_settings.scroll_paginated) - { - if (gui_list->start_item + nb_lines <= gui_list->selected_item) - gui_list->start_item = MIN(bottom, gui_list->selected_item); + if (gui_list->start_item[screen] + nb_lines <= gui_list->selected_item) + gui_list->start_item[screen] = MIN(bottom, gui_list->selected_item); } else { int top_of_screen = gui_list->selected_item + SCROLL_LIMIT - nb_lines; int temp = MAX(0, top_of_screen); - gui_list->start_item = MIN(bottom, temp); + gui_list->start_item[screen] = MIN(bottom, temp); } } - else gui_list->start_item = 0; +} +/* + * Selects an item in the list + * - gui_list : the list structure + * - item_number : the number of the item which will be selected + */ +void gui_synclist_select_item(struct gui_synclist * gui_list, int item_number) +{ + int i; + if( item_number > gui_list->nb_items-1 || item_number < 0 ) + return; + gui_list->selected_item = item_number; + FOR_NB_SCREENS(i) + gui_list_put_selection_on_screen(gui_list, i); } -static void gui_list_select_at_offset(struct gui_list * gui_list, int offset) +static void gui_list_select_at_offset(struct gui_synclist * gui_list, + int offset) { - /* do this here instead of in both select_above and select_below */ - int nb_lines = gui_list->display->nb_lines; - if (SHOW_LIST_TITLE) - nb_lines--; - + int new_selection = gui_list->selected_item + offset; if (gui_list->selected_size > 1) { offset *= gui_list->selected_size; /* always select the first item of multi-line lists */ offset -= offset%gui_list->selected_size; } - if (offset == 0 && global_settings.scroll_paginated && - (gui_list->nb_items > nb_lines)) + if (new_selection >= gui_list->nb_items) { - int bottom = gui_list->nb_items - nb_lines; - gui_list->start_item = MIN(gui_list->selected_item, bottom); + if (gui_list->limit_scroll) + gui_list->selected_item = gui_list->nb_items - 1; + else + gui_list->selected_item = new_selection - gui_list->nb_items; } - else if (offset < 0) - gui_list_select_above(gui_list, -offset, nb_lines); - else - gui_list_select_below(gui_list, offset, nb_lines); + else if (new_selection < 0) + { + if (gui_list->limit_scroll) + gui_list->selected_item = 0; + else + gui_list->selected_item = new_selection + gui_list->nb_items; + } + else if (gui_list->show_selection_marker == false) + { + /* NOTE: this part doesnt work as well as it used to, the problem is + we want to scroll the lists seperatly but we only have one + selected item now, I dont think this is such a big deal though */ + int i, nb_lines, screen_top; + FOR_NB_SCREENS(i) + { + struct screen *display = &screens[i]; + nb_lines = display->nb_lines - SHOW_LIST_TITLE; + if (offset > 0) + { + screen_top = gui_list->nb_items-nb_lines; + if (screen_top < 0) + screen_top = 0; + gui_list->start_item[i] = MIN(screen_top, gui_list->start_item[i] + + gui_list->selected_size); + } + else + { + gui_list->start_item[i] = MAX(0, gui_list->start_item[i] - + gui_list->selected_size); + } + } + return; + } + else gui_list->selected_item += offset; + gui_synclist_select_item(gui_list, gui_list->selected_item); + } /* * Adds an item to the list (the callback will be asked for one more item) * - gui_list : the list structure */ -static void gui_list_add_item(struct gui_list * gui_list) +void gui_synclist_add_item(struct gui_synclist * gui_list) { gui_list->nb_items++; /* if only one item in the list, select it */ @@ -659,58 +618,29 @@ static void gui_list_add_item(struct gui_list * gui_list) * Removes an item to the list (the callback will be asked for one less item) * - gui_list : the list structure */ -static void gui_list_del_item(struct gui_list * gui_list) +void gui_synclist_del_item(struct gui_synclist * gui_list) { + int i; if(gui_list->nb_items > 0) { - gui_textarea_update_nblines(gui_list->display); - int nb_lines = gui_list->display->nb_lines; - - int dist_selected_from_end = gui_list->nb_items - - gui_list->selected_item - 1; - int dist_start_from_end = gui_list->nb_items - - gui_list->start_item - 1; - if(dist_selected_from_end == 0) - { - /* Oops we are removing the selected item, - select the previous one */ + if (gui_list->selected_item == gui_list->nb_items-1) gui_list->selected_item--; + FOR_NB_SCREENS(i) + { + gui_textarea_update_nblines(&screens[i]); + int nb_lines = screens[i].nb_lines; + int dist_start_from_end = gui_list->nb_items + - gui_list->start_item[i] - 1; + + /* scroll the list if needed */ + if( (dist_start_from_end < nb_lines) && (gui_list->start_item[i] != 0) ) + gui_list->start_item[i]--; } gui_list->nb_items--; - - /* scroll the list if needed */ - if( (dist_start_from_end < nb_lines) && (gui_list->start_item != 0) ) - gui_list->start_item--; } } #ifdef HAVE_LCD_BITMAP - -/* - * Makes all the item in the list scroll by one step to the right. - * Should stop increasing the value when reaching the widest item value - * in the list. - */ -static void gui_list_scroll_right(struct gui_list * gui_list) -{ - /* FIXME: This is a fake right boundry limiter. there should be some - * callback function to find the longest item on the list in pixels, - * to stop the list from scrolling past that point */ - gui_list->offset_position+=offset_step; - if (gui_list->offset_position > 1000) - gui_list->offset_position = 1000; -} - -/* - * Makes all the item in the list scroll by one step to the left. - * stops at starting position. - */ -static void gui_list_scroll_left(struct gui_list * gui_list) -{ - gui_list->offset_position-=offset_step; - if (gui_list->offset_position < 0) - gui_list->offset_position = 0; -} void gui_list_screen_scroll_step(int ofs) { offset_step = ofs; @@ -729,14 +659,16 @@ void gui_list_screen_scroll_out_of_view(bool enable) * Set the title and title icon of the list. Setting title to NULL disables * both the title and icon. Use NOICON if there is no icon. */ -static void gui_list_set_title(struct gui_list * gui_list, +void gui_synclist_set_title(struct gui_synclist * gui_list, char * title, enum themable_icons icon) { gui_list->title = title; gui_list->title_icon = icon; if (title) { #ifdef HAVE_LCD_BITMAP - gui_list->display->getstringsize(title, &gui_list->title_width, NULL); + int i; + FOR_NB_SCREENS(i) + screens[i].getstringsize(title, &gui_list->title_width, NULL); #else gui_list->title_width = strlen(title); #endif @@ -745,143 +677,96 @@ static void gui_list_set_title(struct gui_list * gui_list, } } -/* - * Synchronized lists stuffs - */ -void gui_synclist_init( - struct gui_synclist * lists, - list_get_name callback_get_item_name, - void * data, - bool scroll_all, - int selected_size - ) -{ - int i; - FOR_NB_SCREENS(i) - { - gui_list_init(&(lists->gui_list[i]), - callback_get_item_name, - data, scroll_all, selected_size); - gui_list_set_display(&(lists->gui_list[i]), &(screens[i])); - } -} void gui_synclist_set_nb_items(struct gui_synclist * lists, int nb_items) { +#ifdef HAVE_LCD_BITMAP int i; +#endif + lists->nb_items = nb_items; +#ifdef HAVE_LCD_BITMAP FOR_NB_SCREENS(i) { - gui_list_set_nb_items(&(lists->gui_list[i]), nb_items); -#ifdef HAVE_LCD_BITMAP - lists->gui_list[i].offset_position = 0; -#endif + lists->offset_position[i] = 0; } +#endif } int gui_synclist_get_nb_items(struct gui_synclist * lists) { - return gui_list_get_nb_items(&((lists)->gui_list[0])); + return lists->nb_items; } int gui_synclist_get_sel_pos(struct gui_synclist * lists) { - return gui_list_get_sel_pos(&((lists)->gui_list[0])); + return lists->selected_item; } void gui_synclist_set_icon_callback(struct gui_synclist * lists, list_get_icon icon_callback) { - int i; - FOR_NB_SCREENS(i) - { - gui_list_set_icon_callback(&(lists->gui_list[i]), icon_callback); - } + lists->callback_get_item_icon = icon_callback; } void gui_synclist_set_voice_callback(struct gui_synclist * lists, list_speak_item voice_callback) { - gui_list_set_voice_callback(&(lists->gui_list[0]), voice_callback); -} - -void gui_synclist_draw(struct gui_synclist * lists) -{ - int i; - FOR_NB_SCREENS(i) - gui_list_draw(&(lists->gui_list[i])); -} - -void gui_synclist_select_item(struct gui_synclist * lists, int item_number) -{ - int i; - FOR_NB_SCREENS(i) - gui_list_select_item(&(lists->gui_list[i]), item_number); + lists->callback_speak_item = voice_callback; } static void gui_synclist_select_next_page(struct gui_synclist * lists, enum screen_type screen) { - int i; - FOR_NB_SCREENS(i) - gui_list_select_at_offset(&(lists->gui_list[i]), - screens[screen].nb_lines); + gui_list_select_at_offset(lists, screens[screen].nb_lines); } static void gui_synclist_select_previous_page(struct gui_synclist * lists, enum screen_type screen) { - int i; - FOR_NB_SCREENS(i) - gui_list_select_at_offset(&(lists->gui_list[i]), - -screens[screen].nb_lines); -} - -void gui_synclist_add_item(struct gui_synclist * lists) -{ - int i; - FOR_NB_SCREENS(i) - gui_list_add_item(&(lists->gui_list[i])); -} - -void gui_synclist_del_item(struct gui_synclist * lists) -{ - int i; - FOR_NB_SCREENS(i) - gui_list_del_item(&(lists->gui_list[i])); + gui_list_select_at_offset(lists, -screens[screen].nb_lines); } void gui_synclist_limit_scroll(struct gui_synclist * lists, bool scroll) { - int i; - FOR_NB_SCREENS(i) - gui_list_limit_scroll(&(lists->gui_list[i]), scroll); -} - -void gui_synclist_set_title(struct gui_synclist * lists, - char * title, enum themable_icons icon) -{ - int i; - FOR_NB_SCREENS(i) - gui_list_set_title(&(lists->gui_list[i]), title, icon); + lists->limit_scroll = scroll; } #ifdef HAVE_LCD_BITMAP +/* + * Makes all the item in the list scroll by one step to the right. + * Should stop increasing the value when reaching the widest item value + * in the list. + */ static void gui_synclist_scroll_right(struct gui_synclist * lists) { int i; FOR_NB_SCREENS(i) - gui_list_scroll_right(&(lists->gui_list[i])); + { + /* FIXME: This is a fake right boundry limiter. there should be some + * callback function to find the longest item on the list in pixels, + * to stop the list from scrolling past that point */ + lists->offset_position[i]+=offset_step; + if (lists->offset_position[i] > 1000) + lists->offset_position[i] = 1000; + } } +/* + * Makes all the item in the list scroll by one step to the left. + * stops at starting position. + */ static void gui_synclist_scroll_left(struct gui_synclist * lists) { int i; FOR_NB_SCREENS(i) - gui_list_scroll_left(&(lists->gui_list[i])); + { + lists->offset_position[i]-=offset_step; + if (lists->offset_position[i] < 0) + lists->offset_position[i] = 0; + } } #endif /* HAVE_LCD_BITMAP */ static void _gui_synclist_speak_item(struct gui_synclist *lists, bool repeating) { - struct gui_list *l = &lists->gui_list[0]; - list_speak_item *cb = l->callback_speak_item; + list_speak_item *cb = lists->callback_speak_item; if(cb && gui_synclist_get_nb_items(lists) != 0) { int sel = gui_synclist_get_sel_pos(lists); @@ -892,17 +777,17 @@ static void _gui_synclist_speak_item(struct gui_synclist *lists, bool repeating) pending scheduled announcement not yet due: we need to reschedule it. */ if(repeating - || (l->scheduled_talk_tick - && TIME_BEFORE(current_tick, l->scheduled_talk_tick)) - || (l->last_talked_tick - && TIME_BEFORE(current_tick, l->last_talked_tick +HZ/4))) + || (lists->scheduled_talk_tick + && TIME_BEFORE(current_tick, lists->scheduled_talk_tick)) + || (lists->last_talked_tick + && TIME_BEFORE(current_tick, lists->last_talked_tick +HZ/4))) { - l->scheduled_talk_tick = current_tick +HZ/4; + lists->scheduled_talk_tick = current_tick +HZ/4; return; } else { - l->scheduled_talk_tick = 0; /* work done */ - cb(sel, l->data); - l->last_talked_tick = current_tick; + lists->scheduled_talk_tick = 0; /* work done */ + cb(sel, lists->data); + lists->last_talked_tick = current_tick; } } } @@ -1014,7 +899,6 @@ bool gui_synclist_do_button(struct gui_synclist * lists, #ifdef HAVE_LCD_BITMAP static bool scrolling_left = false; #endif - int i; #ifdef HAVE_SCROLLWHEEL int next_item_modifier = button_apply_acceleration(get_action_data()); @@ -1086,8 +970,7 @@ bool gui_synclist_do_button(struct gui_synclist * lists, #endif case ACTION_STD_PREV: case ACTION_STD_PREVREPEAT: - FOR_NB_SCREENS(i) - gui_list_select_at_offset(&(lists->gui_list[i]), -next_item_modifier); + gui_list_select_at_offset(lists, -next_item_modifier); #ifndef HAVE_SCROLLWHEEL if (queue_count(&button_queue) < FRAMEDROP_TRIGGER) #endif @@ -1101,8 +984,7 @@ bool gui_synclist_do_button(struct gui_synclist * lists, case ACTION_STD_NEXT: case ACTION_STD_NEXTREPEAT: - FOR_NB_SCREENS(i) - gui_list_select_at_offset(&(lists->gui_list[i]), next_item_modifier); + gui_list_select_at_offset(lists, next_item_modifier); #ifndef HAVE_SCROLLWHEEL if (queue_count(&button_queue) < FRAMEDROP_TRIGGER) #endif @@ -1125,7 +1007,7 @@ bool gui_synclist_do_button(struct gui_synclist * lists, keymaps as a repeated button press (the same as the repeated ACTION_TREE_PGLEFT) with the pre condition being the non-repeated button press */ - if (lists->gui_list[0].offset_position == 0) + if (lists->offset_position[0] == 0) { scrolling_left = false; *actionptr = ACTION_STD_CANCEL; @@ -1133,7 +1015,7 @@ bool gui_synclist_do_button(struct gui_synclist * lists, } *actionptr = ACTION_TREE_PGLEFT; case ACTION_TREE_PGLEFT: - if(!scrolling_left && (lists->gui_list[0].offset_position == 0)) + if(!scrolling_left && (lists->offset_position[0] == 0)) { *actionptr = ACTION_STD_CANCEL; return false; @@ -1144,7 +1026,6 @@ bool gui_synclist_do_button(struct gui_synclist * lists, skipping to root */ return true; #endif - /* for pgup / pgdown, we are obliged to have a different behaviour depending * on the screen for which the user pressed the key since for example, remote * and main screen doesn't have the same number of lines */ @@ -1180,8 +1061,8 @@ bool gui_synclist_do_button(struct gui_synclist * lists, } return true; } - if(lists->gui_list[0].scheduled_talk_tick - && TIME_AFTER(current_tick, lists->gui_list[0].scheduled_talk_tick)) + if(lists->scheduled_talk_tick + && TIME_AFTER(current_tick, lists->scheduled_talk_tick)) /* scheduled postponed item announcement is due */ _gui_synclist_speak_item(lists, false); return false; @@ -1191,9 +1072,9 @@ int list_do_action_timeout(struct gui_synclist *lists, int timeout) /* Returns the lowest of timeout or the delay until a postponed scheduled announcement is due (if any). */ { - if(lists->gui_list[0].scheduled_talk_tick) + if(lists->scheduled_talk_tick) { - long delay = lists->gui_list[0].scheduled_talk_tick -current_tick +1; + long delay = lists->scheduled_talk_tick -current_tick +1; /* +1 because the trigger condition uses TIME_AFTER(), which is implemented as strictly greater than. */ if(delay < 0) diff --git a/apps/gui/list.h b/apps/gui/list.h index 742e30dbaa..3bd3d25c49 100644 --- a/apps/gui/list.h +++ b/apps/gui/list.h @@ -82,7 +82,7 @@ typedef int list_speak_item(int selected_item, void * data); typedef int list_get_color(int selected_item, void * data); #endif -struct gui_list +struct gui_synclist { /* defines wether the list should stop when reaching the top/bottom * or should continue (by going to bottom/top) */ @@ -93,15 +93,15 @@ struct gui_list int nb_items; int selected_item; - int start_item; /* the item that is displayed at the top of the screen */ + int start_item[NB_SCREENS]; /* the item that is displayed at the top of the screen */ /* the number of lines that are selected at the same time */ int selected_size; /* These are used to calculate how much of the screen content we need to redraw. */ int last_displayed_selected_item; - int last_displayed_start_item; + int last_displayed_start_item[NB_SCREENS]; #ifdef HAVE_LCD_BITMAP - int offset_position; /* the list's screen scroll placement in pixels */ + int offset_position[NB_SCREENS]; /* the list's screen scroll placement in pixels */ #endif /* Cache the width of the title string in pixels/characters */ int title_width; @@ -111,7 +111,6 @@ struct gui_list list_get_name *callback_get_item_name; list_speak_item *callback_speak_item; - struct screen * display; /* The data that will be passed to the callback function YOU implement */ void * data; /* The optional title, set to NULL for none */ @@ -195,19 +194,6 @@ extern void gui_list_screen_scroll_out_of_view(bool enable); #define gui_list_limit_scroll(gui_list, scroll) \ (gui_list)->limit_scroll=scroll -/* - * This part handles as many lists as there are connected screens - * (the api is similar to the ones above) - * The lists on the screens are synchronized ; - * theirs items and selected items are the same, but of course, - * they can be displayed on screens with different sizes - * The final aim is to let the programmer handle many lists in one - * function call and make its code independant from the number of screens - */ -struct gui_synclist -{ - struct gui_list gui_list[NB_SCREENS]; -}; extern void gui_synclist_init( struct gui_synclist * lists, diff --git a/apps/menus/main_menu.c b/apps/menus/main_menu.c index 93d937e6db..a0786f9bc6 100644 --- a/apps/menus/main_menu.c +++ b/apps/menus/main_menu.c @@ -361,7 +361,7 @@ static int info_action_callback(int action, struct gui_synclist *lists) ) { #ifndef SIMULATOR - struct info_data *info = (struct info_data *)lists->gui_list[SCREEN_MAIN].data; + struct info_data *info = (struct info_data *)lists->data; info->new_data = true; gui_syncsplash(0, ID2P(LANG_SCANNING_DISK)); fat_recalc_free(IF_MV(0)); diff --git a/apps/tree.c b/apps/tree.c index b800833c7d..35fa05364a 100644 --- a/apps/tree.c +++ b/apps/tree.c @@ -330,7 +330,7 @@ void tree_gui_init(void) gui_synclist_set_voice_callback(&tree_lists, tree_voice_cb); gui_synclist_set_icon_callback(&tree_lists, &tree_get_fileicon); #ifdef HAVE_LCD_COLOR - gui_list_set_color_callback(&tree_lists.gui_list[SCREEN_MAIN], + gui_list_set_color_callback(&tree_lists, &tree_get_filecolor); #endif }