diff options
author | Peter D'Hoye <peter.dhoye@gmail.com> | 2008-06-28 20:45:21 +0000 |
---|---|---|
committer | Peter D'Hoye <peter.dhoye@gmail.com> | 2008-06-28 20:45:21 +0000 |
commit | 205f3df7816a1eea9c812ea285d74a4f8ecfad2a (patch) | |
tree | 356be7b807a4407b7e243ec57da4d5068fe09ab1 /apps/gui/color_picker.c | |
parent | 3d240f1e2a34e616c2aba22b58ea78de7f277127 (diff) | |
download | rockbox-205f3df7816a1eea9c812ea285d74a4f8ecfad2a.tar.gz rockbox-205f3df7816a1eea9c812ea285d74a4f8ecfad2a.zip |
Remove a viewport ambiguity by changing the screens width/heigth members into lcdwidth/lcdheight. Normal usercode should always use getwidth()/getheigth() as that returns the viewport width/height. Fixes issues that would have appeared in many places when introducing viewports with sizes != lcd sizes.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@17857 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'apps/gui/color_picker.c')
-rw-r--r-- | apps/gui/color_picker.c | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/apps/gui/color_picker.c b/apps/gui/color_picker.c index 239a8b4c81..4ac4fa7fff 100644 --- a/apps/gui/color_picker.c +++ b/apps/gui/color_picker.c | |||
@@ -158,11 +158,11 @@ static void draw_screen(struct screen *display, char *title, | |||
158 | enough to display the selected slider - calculate total height | 158 | enough to display the selected slider - calculate total height |
159 | of display with three sliders present */ | 159 | of display with three sliders present */ |
160 | display_three_rows = | 160 | display_three_rows = |
161 | display->height >= MARGIN_TOP + | 161 | display->getheight() >= MARGIN_TOP + |
162 | display->char_height*4 + /* Title + 3 sliders */ | 162 | display->char_height*4 + /* Title + 3 sliders */ |
163 | TITLE_MARGIN_BOTTOM + | 163 | TITLE_MARGIN_BOTTOM + |
164 | SELECTOR_TB_MARGIN*6 + /* 2 margins/slider */ | 164 | SELECTOR_TB_MARGIN*6 + /* 2 margins/slider */ |
165 | MARGIN_BOTTOM; | 165 | MARGIN_BOTTOM; |
166 | 166 | ||
167 | /* Figure out widest label character in case they vary - | 167 | /* Figure out widest label character in case they vary - |
168 | this function assumes labels are one character */ | 168 | this function assumes labels are one character */ |
@@ -178,13 +178,13 @@ static void draw_screen(struct screen *display, char *title, | |||
178 | /* Draw title string */ | 178 | /* Draw title string */ |
179 | set_drawinfo(display, DRMODE_SOLID, text_color, background_color); | 179 | set_drawinfo(display, DRMODE_SOLID, text_color, background_color); |
180 | display->getstringsize(title, &x, &y); | 180 | display->getstringsize(title, &x, &y); |
181 | display->putsxy((display->width - x) / 2, MARGIN_TOP, title); | 181 | display->putsxy((display->getwidth() - x) / 2, MARGIN_TOP, title); |
182 | 182 | ||
183 | /* Get slider positions and top starting position */ | 183 | /* Get slider positions and top starting position */ |
184 | text_top = MARGIN_TOP + y + TITLE_MARGIN_BOTTOM + SELECTOR_TB_MARGIN; | 184 | text_top = MARGIN_TOP + y + TITLE_MARGIN_BOTTOM + SELECTOR_TB_MARGIN; |
185 | slider_left = MARGIN_LEFT + SELECTOR_WIDTH + SELECTOR_LR_MARGIN + | 185 | slider_left = MARGIN_LEFT + SELECTOR_WIDTH + SELECTOR_LR_MARGIN + |
186 | max_label_width + SLIDER_MARGIN_LEFT; | 186 | max_label_width + SLIDER_MARGIN_LEFT; |
187 | slider_width = display->width - slider_left - SLIDER_MARGIN_RIGHT - | 187 | slider_width = display->getwidth() - slider_left - SLIDER_MARGIN_RIGHT - |
188 | display->char_width*2 - SELECTOR_LR_MARGIN - SELECTOR_WIDTH - | 188 | display->char_width*2 - SELECTOR_LR_MARGIN - SELECTOR_WIDTH - |
189 | MARGIN_RIGHT; | 189 | MARGIN_RIGHT; |
190 | 190 | ||
@@ -208,7 +208,7 @@ static void draw_screen(struct screen *display, char *title, | |||
208 | /* Draw solid bar selection bar */ | 208 | /* Draw solid bar selection bar */ |
209 | display->fillrect(0, | 209 | display->fillrect(0, |
210 | text_top - SELECTOR_TB_MARGIN, | 210 | text_top - SELECTOR_TB_MARGIN, |
211 | display->width, | 211 | display->getwidth(), |
212 | display->char_height + | 212 | display->char_height + |
213 | SELECTOR_TB_MARGIN*2); | 213 | SELECTOR_TB_MARGIN*2); |
214 | 214 | ||
@@ -225,8 +225,8 @@ static void draw_screen(struct screen *display, char *title, | |||
225 | SELECTOR_HEIGHT) / 2; | 225 | SELECTOR_HEIGHT) / 2; |
226 | screen_put_iconxy(display, MARGIN_LEFT, top, Icon_Cursor); | 226 | screen_put_iconxy(display, MARGIN_LEFT, top, Icon_Cursor); |
227 | screen_put_iconxy(display, | 227 | screen_put_iconxy(display, |
228 | display->width - MARGIN_RIGHT - | 228 | display->getwidth() - MARGIN_RIGHT - |
229 | get_icon_width(display->screen_type), | 229 | get_icon_width(display->screen_type), |
230 | top, Icon_Cursor); | 230 | top, Icon_Cursor); |
231 | } | 231 | } |
232 | 232 | ||
@@ -282,9 +282,9 @@ static void draw_screen(struct screen *display, char *title, | |||
282 | /* Display color swatch on color screens only */ | 282 | /* Display color swatch on color screens only */ |
283 | int left = MARGIN_LEFT + SELECTOR_WIDTH + SELECTOR_LR_MARGIN; | 283 | int left = MARGIN_LEFT + SELECTOR_WIDTH + SELECTOR_LR_MARGIN; |
284 | int top = text_top + SWATCH_TOP_MARGIN; | 284 | int top = text_top + SWATCH_TOP_MARGIN; |
285 | int width = display->width - left - SELECTOR_LR_MARGIN - | 285 | int width = display->getwidth() - left - SELECTOR_LR_MARGIN - |
286 | SELECTOR_WIDTH - MARGIN_RIGHT; | 286 | SELECTOR_WIDTH - MARGIN_RIGHT; |
287 | int height = display->height - top - MARGIN_BOTTOM; | 287 | int height = display->getheight() - top - MARGIN_BOTTOM; |
288 | 288 | ||
289 | /* Only draw if room */ | 289 | /* Only draw if room */ |
290 | if (height >= display->char_height + 2) | 290 | if (height >= display->char_height + 2) |
@@ -314,11 +314,11 @@ static void draw_screen(struct screen *display, char *title, | |||
314 | display->getstringsize(buf, &x, &y); | 314 | display->getstringsize(buf, &x, &y); |
315 | i = text_top + SWATCH_TOP_MARGIN; | 315 | i = text_top + SWATCH_TOP_MARGIN; |
316 | 316 | ||
317 | if (i + y <= display->height - MARGIN_BOTTOM) | 317 | if (i + y <= display->getheight() - MARGIN_BOTTOM) |
318 | { | 318 | { |
319 | set_drawinfo(display, DRMODE_SOLID, text_color, background_color); | 319 | set_drawinfo(display, DRMODE_SOLID, text_color, background_color); |
320 | x = (display->width - x) / 2; | 320 | x = (display->getwidth() - x) / 2; |
321 | y = (i + display->height - MARGIN_BOTTOM - y) / 2; | 321 | y = (i + display->getheight() - MARGIN_BOTTOM - y) / 2; |
322 | display->putsxy(x, y, buf); | 322 | display->putsxy(x, y, buf); |
323 | } | 323 | } |
324 | } | 324 | } |