From a7c8c71dad328fcd97a42667b824d272fb46d993 Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Thu, 12 Oct 2006 00:50:13 +0000 Subject: Corrected the calculation to determine to display three sliders or one in the colorpicker. Same wonderment as before... git-svn-id: svn://svn.rockbox.org/rockbox/trunk@11197 a1c6a512-1295-4272-9138-f99709370657 --- apps/gui/color_picker.c | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/apps/gui/color_picker.c b/apps/gui/color_picker.c index 76c47fe376..719112df87 100644 --- a/apps/gui/color_picker.c +++ b/apps/gui/color_picker.c @@ -145,14 +145,18 @@ static void draw_screen(struct screen *display, char *title, background_color = display->get_background(); } - /* Find out if there's enought room for three slider or just - enought to display the selected slider */ - display_three_rows = display->height >= display->char_height*4 + - MARGIN_TOP + - TITLE_MARGIN_BOTTOM + - MARGIN_BOTTOM; - - /* Figure out widest label character in case they vary */ + /* Find out if there's enough room for three sliders or just + enough to display the selected slider - calculate total height + of display with three sliders present */ + display_three_rows = + display->height >= MARGIN_TOP + + display->char_height*4 + /* Title + 3 sliders */ + TITLE_MARGIN_BOTTOM + + SELECTOR_TB_MARGIN*6 + /* 2 margins/slider */ + MARGIN_BOTTOM; + + /* Figure out widest label character in case they vary - + this function assumes labels are one character */ for (i = 0, max_label_width = 0; i < 3; i++) { buf[0] = str(LANG_COLOR_RGB_LABELS)[i]; @@ -232,7 +236,7 @@ static void draw_screen(struct screen *display, char *title, set_drawinfo(display, mode, fg, bg); - /* Draw label - assumes labels are one character */ + /* Draw label */ buf[0] = str(LANG_COLOR_RGB_LABELS)[i]; buf[1] = '\0'; display->putsxy(slider_left - display->char_width - -- cgit v1.2.3