- Status Closed
- Percent Complete
- Task Type Bugs
- Category User Interface → Themes
-
Assigned To
Jonathan Gordon - Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Opened by Thomas Martitz - 2009-09-08
Last edited by Thomas Martitz - 2009-09-08
FS#10595 - First skin_line in a conditional vp is is also part of the previous vp
So, I noticed a visual glitch with the Azure Ultimate theme (http://themes.rockbox.org/index.php?target=e200) where a grey bar would show in the progressbar viewport (which is defined above). See http://imagebin.org/62908 (this is actually with the custom statusbar patch, but the same problem happens in SVN with the non-custom-sb version of the theme).
It turns out that it was caused by r22618.
I moved the progressbar viewport away, which made that grey bar show up in the viewport that was above progressbar one. I also saw the grey bar is actually the top few pixels of a the keypad locked image (the keypad locked image shows instead of “Now Playing” at the very top if hold is on).
This means: The this image is in a conditional viewport, and shows up in the viewport defined above. It also shows in the viewport it’s defined in.
I’ve attached a patch which seems to fix it (the line is suspicious because an index should be always num_* -1 (if index is starting with 0, and num_ with 1) ? ). But I have no idea if that’s the proper fix. I also have no idea why it apparently only happens with condition viewports, but that may just be my theme which triggers it.
cond-vp-fix.diff
(0.6 KiB)
2009-09-08 16:35
Reason for closing: Fixed
Additional comments about closing:
Fixed in r22662.