Rockbox

Tasklist

FS#2994 - Scrolling Margins for new CVS

Attached to Project: Rockbox
Opened by takka (tfact) - Tuesday, 07 February 2006, 06:19 GMT
Last edited by Matthias Mohr (aka Massa) (mmohr) - Sunday, 02 April 2006, 18:04 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The patch adds the possibility to restrict the width of a (scrolling) line.
It enables you to position the scroll text inside the line and give him a
starting and ending position.
It extents the %s WPS tag to the following:
%s%m|x1|x2|
where x1 is the x-position where the line text starts and x2 is the
ending x-position of the scroll text (actually, when I look at the
code, it should also work for nonscrolling text...)
Only one margin is allowed per line.

BTW, I assume this patch will never find it's way into the CVS.
The devs plan something more generous, called "viewports".
The plan is to include it in v3.1 - regarding the release plan it
will be somewhere between now and (latest) November 2006.

Until the viewports find it's way into CVS this patch here seems
to be useful for a lot of people...
(and it is also the base for other patches)

You may find the patch compatible to the current CVS always
at the end of the comments to this task...
This task depends upon

Closed by  Ben Basha (paprica)
Monday, 03 April 2006, 12:36 GMT
Reason for closing:  Duplicate
Additional comments about closing:  we are returning to the orginal task
http://www.rockbox.org/tracker/task/2954
Comment by Matthias Mohr (aka Massa) (mmohr) - Saturday, 11 March 2006, 20:58 GMT
Here is the patch for today's CVS (20060311)

Hopefully the changed patch will work until it finally find it's way into the CVS ;)
Comment by Matthias Mohr (aka Massa) (mmohr) - Friday, 24 March 2006, 22:26 GMT
Here is another sync to today's CVS (2006-03-24)

Although it seems it'll never find it's way into CVS...
Comment by Rob (biffhero) - Saturday, 25 March 2006, 05:42 GMT
scroll-margins_20060324.patch applied to the cvs download I have from 2006-03-24_22:02 However, it does not build.

CC debug_menu.c
debug_menu.c: In function `dbg_ports':
debug_menu.c:1042: error: too few arguments to function `lcd_setmargins'
make[1]: *** [/home/rob/ipod-linux/rockbox/cvs/src/rockbox-bleeding/build/apps/debug_menu.o] Error 1
make: *** [all] Error 2

I looked at your patch, and changed line 1042 to be -> lcd_setmargins(0, LCD_WIDTH, 0);

And it at least builds now. :-)

Hope this gets into CVS soon!
Comment by Rob (biffhero) - Saturday, 25 March 2006, 06:36 GMT
It built just fine, and my iPod 5G runs ok. I don't know, however, what this patch is supposed to do, so I can't tell you if I have tested exactly this code.
Comment by Alexander Bondar (davinci) - Saturday, 25 March 2006, 10:14 GMT
This patch will never get into CVS, so the developers. They want to make something like containers which can get positioned at any position on the display. Sorry, I don't know the name of it anymore.
But as long there is no alternative for the scroll margins patch I will use it.
Comment by Matthias Mohr (aka Massa) (mmohr) - Saturday, 25 March 2006, 23:48 GMT
Rob, you were right. It seems I missed that line - and a few others ;)
I added an updated patch which also fixes some small bugs I found.

I also updated the description of the patch above...

And yes, I also doubt it will ever be put into CVS.
The devs plan something more generous, called "viewports".
The plan is to include it in v3.1 - regarding the release plan it'll
be (latest) November 2006.

So until then this patch here is needed for a lot of people...
(and is also the base for other patches)
Comment by Nicolas Pennequin (nicolas_p) - Wednesday, 29 March 2006, 14:58 GMT
Synched with current CVS (2006-03-29)
Comment by Nicolas Pennequin (nicolas_p) - Wednesday, 29 March 2006, 15:49 GMT
Fixed a small problem in the previous patch
Comment by Matthias Mohr (aka Massa) (mmohr) - Sunday, 02 April 2006, 18:00 GMT
Here is another sync to the current CVS (2006-04-02)

Loading...