diff options
| author | Nicolas Pennequin <nicolas.pennequin@free.fr> | 2007-04-09 14:06:38 +0000 |
|---|---|---|
| committer | Nicolas Pennequin <nicolas.pennequin@free.fr> | 2007-04-09 14:06:38 +0000 |
| commit | cda89411b0baa33cdec52099968fac0985846263 (patch) | |
| tree | b77e3b700651668d9596c0f0d3a482ddfd4c8837 /apps/gui | |
| parent | 5764ff571a9f1c23171c1caa858c229770a049c7 (diff) | |
| download | rockbox-cda89411b0baa33cdec52099968fac0985846263.zip rockbox-cda89411b0baa33cdec52099968fac0985846263.tar.gz rockbox-cda89411b0baa33cdec52099968fac0985846263.tar.bz2 rockbox-cda89411b0baa33cdec52099968fac0985846263.tar.xz | |
Accept FS#7000 by Dave Hooper: Fix scrolling line behaviour for dynamic tags (e.g. when a line with info about the next track didn't scroll) and fix some incorrect refreshing flags.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@13081 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'apps/gui')
| -rw-r--r-- | apps/gui/gwps-common.c | 11 | ||||
| -rw-r--r-- | apps/gui/wps_parser.c | 4 |
2 files changed, 11 insertions, 4 deletions
diff --git a/apps/gui/gwps-common.c b/apps/gui/gwps-common.c index 62a068f..cc1f13a 100644 --- a/apps/gui/gwps-common.c +++ b/apps/gui/gwps-common.c @@ -1883,8 +1883,15 @@ bool gui_wps_refresh(struct gui_wps *gwps, if (update_line) { - /* calculate alignment and draw the strings */ - write_line(display, &align, line, flags & WPS_REFRESH_SCROLL); + if (flags & WPS_REFRESH_SCROLL) + { + /* if the line is a scrolling one we don't want to update + too often, so that it has the time to scroll */ + if (refresh_mode & WPS_REFRESH_SCROLL) + write_line(display, &align, line, true); + } + else + write_line(display, &align, line, false); } } diff --git a/apps/gui/wps_parser.c b/apps/gui/wps_parser.c index 3a85b5f..8d917cd 100644 --- a/apps/gui/wps_parser.c +++ b/apps/gui/wps_parser.c @@ -191,13 +191,13 @@ static const struct wps_tag all_tags[] = { { WPS_TOKEN_METADATA_TRACK_TITLE, "it", WPS_REFRESH_STATIC, NULL }, { WPS_TOKEN_METADATA_VERSION, "iv", WPS_REFRESH_STATIC, NULL }, { WPS_TOKEN_METADATA_YEAR, "iy", WPS_REFRESH_STATIC, NULL }, - { WPS_TOKEN_METADATA_COMMENT, "iC", WPS_REFRESH_DYNAMIC, NULL }, + { WPS_TOKEN_METADATA_COMMENT, "iC", WPS_REFRESH_STATIC, NULL }, /* next metadata */ { WPS_TOKEN_METADATA_ARTIST, "Ia", WPS_REFRESH_DYNAMIC, NULL }, { WPS_TOKEN_METADATA_COMPOSER, "Ic", WPS_REFRESH_DYNAMIC, NULL }, { WPS_TOKEN_METADATA_ALBUM, "Id", WPS_REFRESH_DYNAMIC, NULL }, - { WPS_TOKEN_METADATA_ALBUM_ARTIST, "IA", WPS_REFRESH_STATIC, NULL }, + { WPS_TOKEN_METADATA_ALBUM_ARTIST, "IA", WPS_REFRESH_DYNAMIC, NULL }, { WPS_TOKEN_METADATA_GENRE, "Ig", WPS_REFRESH_DYNAMIC, NULL }, { WPS_TOKEN_METADATA_TRACK_NUMBER, "In", WPS_REFRESH_DYNAMIC, NULL }, { WPS_TOKEN_METADATA_TRACK_TITLE, "It", WPS_REFRESH_DYNAMIC, NULL }, |