Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#6747 : y-margin reset in gui_list_draw

Rockbox

Tasklist

FS#6747 - y-margin reset in gui_list_draw

Attached to Project: Rockbox
Opened by Timo Horstschäfer (x1jmp) - Tuesday, 06 March 2007, 21:26 GMT
Last edited by Jonathan Gordon (jdgordon) - Thursday, 24 April 2008, 10:42 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System Another
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

When calling a menu function, the y-margin got changed, but was not reset after the function returned.

The patch fixes this behaviour and removes all unnecessary calls to lcd_setmargins from the plugins.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Thursday, 24 April 2008, 10:42 GMT
Reason for closing:  Out of Date
Additional comments about closing:  should be all fixed (?) now
Comment by Jonathan Gordon (jdgordon) - Sunday, 11 March 2007, 08:03 GMT
Damn, reopening because this caused a nasty scroll bug, but this should be done..
Comment by Timo Horstschäfer (x1jmp) - Sunday, 11 March 2007, 11:40 GMT
Task  FS#6796  seems to have fixed the scroll bug.
Comment by Jonathan Gordon (jdgordon) - Sunday, 11 March 2007, 13:05 GMT
this was discussed in IRC today and it was bassically decided that storing the x/y pixel value for a scroll line was hacky and not really a good idea, so instead the list widget will be stupid and should be setting the y margin back to old margin when its exited (which isnt know :p ) so we leave it as it is....
Comment by Max Weninger (maxwen) - Tuesday, 20 March 2007, 07:34 GMT
ok :)

Loading...