Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface → Themes
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Ben Basha - 2006-01-28
Last edited by Paul Louden - 2007-11-14

FS#2954 - Scrolling text margin for the wps

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.

Thanks Massa for the description

Closed by  Paul Louden
2007-11-14 20:02
Reason for closing:  Rejected
Additional comments about closing:  

The desired parts of this task have been accepted, and the remainder is rejected, to be done via viewports instead.

Any patch that seeks acceptance should work in this direction and be a new task.

Ben Basha commented on 2006-01-28 02:34

i found a bug

dont download it, i will upload new version in the noon

Ben Basha commented on 2006-01-28 10:23

the bug fixed and now you can use also %ar & %ac

if you want a regular margin after a irregular one,
just use %m without nothing

Ben Basha commented on 2006-01-28 11:56

work with the archos recorder too

Anonymous Submitter commented on 2006-02-01 19:41

iuoçiuç

Ben Basha commented on 2006-02-06 17:21

version 2, work with all the targets… CVS 2006-02-06

Ben Basha commented on 2006-02-06 20:53

buggy, dont download.

takka commented on 2006-02-07 01:11

old version for CVS 0207
http://takka.style.coocan.jp/wiki/?
plugin=attach&pcmd=open&file=scroll-
margins.patch&refer=RockBox

Nicolas Pennequin commented on 2006-02-17 19:34

i synched this patch with the latest CVS (17.02.06). get it
here :
http://nicolas.pennequin.free.fr/scroll_margins.patch

needleboy commented on 2006-02-19 09:31

nicolas_p's version doesn't compile.
neither does Paprica's orginal.

Ben Basha commented on 2006-04-03 12:37

Massa version, synced with the cvs

Matthias Mohr (aka Massa) commented on 2006-04-24 11:31

sync with today's CVS (20060424):

James Wilson commented on 2006-05-12 23:53

Can we get another sync please with the latest CVS I'm getting HUNK errors 1 out of 9

Matthias Mohr (aka Massa) commented on 2006-05-15 15:41

Sorry, I don't have time atm. - so somebody else has to do it.
But beware, I think it's not only synching - there has to be some rework to make it properly work again… (I tried it a week ago and it had some weird effects…)

Chris Banes commented on 2006-05-29 08:51

Here's a resync made by P.I. Julius

Nicolas Pennequin commented on 2006-06-15 17:14

Sync to current CVS

JP commented on 2006-07-25 14:18

Would someone be so kind to sync it to a recent cvs? since I can't get it to work properly myself. Thanks in advance.

JP commented on 2006-07-25 20:35

My request is already fulfilled thanks to drippydonut @ misticriver

Max Weninger commented on 2006-08-12 14:52

Updated patch to work agains CVS HEAD from 12.08.2006

Norbert Preining commented on 2006-08-24 07:47

Hi Max!
The recent changes in cvs (23.8,20:07,Mark Arigo):

    Allow scrolling lines with different x-margins. The margin at the time of the call 
    to puts_scroll will be used as the margin for that line.

have broken the last patch. Does this mean that a similar functionality will be included in CVS soon, or can we adjust the patch to work with the new CVS code.

Thanks a lot and all the best

Norbert

Max Weninger commented on 2006-08-24 11:35

I am still at CVS from 20060820 where the patch still applies
Will check with CVS HEAD asap

Max Weninger commented on 2006-08-24 21:50

IHMO the change is not changing this patch
just the order of two lines has switched

Updated patch to work against CVS from 20060824

Norbert Preining commented on 2006-08-25 06:06

Hi Max!
Thanks a lot. The patch does work after some small changes:

  drivers/lcd-h100-remote.c:1398: error: `xmargin' undeclared (first use in this function)

I changed it to leftmargin. And

  debug_menu.c:1240: error: too few arguments to function `lcd_setmargins'

there is a lcd_setmargins(0,0) left over, I added in the middle the LCD_WIDTH. This way it did compile.

I attach a fixed patch against current cvs for those who want to have it.

Thanks a lot and all the best

Norbert

Max Weninger commented on 2006-08-25 08:41

Ooops!

I only tried it for the simulator and x5 :-(

Thanks for fixing it

Nicolas Pennequin commented on 2006-08-29 02:59

updated for current CVS.

Matthias Mohr (aka Massa) commented on 2006-09-04 21:27

I currently notice a problem when combining this patch with the albumart patch ( FS#3045 ):
I tried to use margins with a bitmap right to it (the margins starts at x=1 and ends one
pixel before the bitmap begins)
If a line left of the bitmap starts to scroll, the line flickers inside the bitmap…

Any idea what can cause this behaviour?

Jon commented on 2006-10-09 15:34

patching file apps/plugin.c
Hunk #1 succeeded at 560 (offset -1 lines).
Hunk #2 succeeded at 574 (offset -1 lines).
Hunk #3 FAILED at 600.
Hunk #4 FAILED at 610.
2 out of 4 hunks FAILED – saving rejects to file apps/plugin.c.rej

patching file apps/sound_menu.c
Hunk #1 succeeded at 742 (offset 1 line).
Hunk #2 succeeded at 790 (offset 1 line).
Hunk #3 FAILED at 1041.
1 out of 3 hunks FAILED – saving rejects to file apps/sound_menu.c.rej

Paul van der Heu commented on 2006-10-09 16:20

And your point is?

Dave Chapman commented on 2006-10-09 19:44

The point appears to be that the patch is out of sync with CVS and needs updating.

Jon commented on 2006-10-10 19:14
Matthias Mohr (aka Massa) commented on 2006-10-23 08:40

Small bugfix (old declaration in plugin.h and wrong function calls in viewer and solitaire-plugins)
and sync to today's CVS

Matthias Mohr (aka Massa) commented on 2006-10-25 08:40

sync to today's CVS

Nicolas Pennequin commented on 2006-11-08 21:18

sync to current CVS

Nicolas Pennequin commented on 2006-11-14 14:42
Johannes Voggenthaler commented on 2006-12-11 14:14

seems to be out of sync again.

Nicolas Pennequin commented on 2007-01-05 19:31

Sync to today's CVS

Nathan Hale commented on 2007-02-11 20:49

The patch applied successfully, but I get the following errors when trying to compile for the Gigabeat F/X port 20070210-build.

debug_menu.c: In function 'dbg_lcd_power_off':
debug_menu.c:2069: error: too few arguments to function 'lcd_setmargins'
debug_menu.c: In function 'dbg_buttonlights':
debug_menu.c:2111: error: too few arguments to function 'lcd_setmargins'

I am able to compile a clean build and also after adding the album art patch.

Thanks for any help.

Nicolas Pennequin commented on 2007-02-12 21:04

This one should work.

Chris commented on 2007-02-13 12:10

Breaks the 4G Grayscale and iPod Mini 2G builds, seems to be okay with 5G and Nano (problem with b/w targets?)

Nicolas Pennequin commented on 2007-02-13 12:44

Better now ? :)

Chris commented on 2007-02-13 20:33

Cheers ^^

Chris commented on 2007-02-14 10:57

Don't think this is quite working on the 5G, for example,

%s%m|18|209|%al%?ia<%ia|%?d2<%d2|(Artist Unknown)»

the |18|209| actually come up on screen?

Derek Van Ittersum commented on 2007-02-19 15:28

Hi, great patch. Recent changes to bookmark.c seem to have broken it though. It patches fine, but when I try to compile, I get the following error:

bookmark.c: In function 'select_bookmark':
bookmark.c:553: error: too few arguments to function 'screens[i]setmargins'

I don't get the error in compiling when I remove the patch. I'm compiling for a Gigabeat F.

Thanks for any help.

Paul van der Heu commented on 2007-02-19 17:26

edit the line to include 'screens[i].width' as second to last argument

So it reads: screens[i].setmargins(XXXX, screens[i].width, XXXX);

Bobby Graese commented on 2007-03-09 02:35

Just a question… will this ever be committed? The progress shows 100% complete, and the last "small bugfix" was on Monday, 23 October 2006… Are there any known bugs left? What are they?

Dave Hooper commented on 2007-03-10 17:33

Sync'ed to latest svn

Marc R. commented on 2007-03-10 20:10

You patch is for revision 12701
Since revision 12704 this patch is not working due to changes in the files list.c and solitaire.c

Dave Hooper commented on 2007-03-11 14:13

Don't worry about that, those changes to list.c and solitaire.c were reverted in svn anyway (as 12719). I just tried repatching latest svn (12723) and it worked fine.

Dan Train commented on 2007-03-12 10:10

Hello, I am new to this lark and am having some trouble using your patch. This is probably not the place to ask but ohwell. When I try to patch and compile using cygwin from the latest build for my iRiver H320 I get the following error:

CC drivers/lcd-remote-1bit-v.c
drivers/lcd-remote-1bit-v.c:82: error: conflicting types for 'lcd_remote_setmargins'
export/lcd-remote.h:145: error: previous declaration of 'lcd_remote_setmargins' was here
drivers/lcd-remote-1bit-v.c:82: error: conflicting types for 'lcd_remote_setmargins'
export/lcd-remote.h:145: error: previous declaration of 'lcd_remote_setmargins' was here
make[1]: * [dir] Error 1
make:
* [all] Error 2

This is doubly annoying since I don't own an LCD remote so really don't care what is displayed on it!

Marc R. commented on 2007-03-12 10:42

I have the following code in my wps-file:
%s%m|16|144|%?it<%it|%d2>
%s%m|16|144|%alNext: %?It<%It|%Fn>

The first line scrolls to the end of the text, stops and scrolls back to the beginning.
In the second line it seems the text will get copied several time.
Like:
strcpy(showntext, "Test ");
for(a=0;a<UNKNOWN_NUMBER;a++)

strcat(showntext, "Test ");

⇒ showntext="Test Test Test Test Test Test Test" (or something like that).
The result is, that the text keeps scrolling without changing direction.
Is this a bug or a feature?

Chris commented on 2007-03-12 10:45

I'm pretty sure its supposed to keep scrolling one way. The 'back and forth' scroll is another FS patch, can't remember which number

Chris Banes commented on 2007-03-12 10:50

Chris,

The scrolling does change direction in this patch, at least it should :D

Chris commented on 2007-03-12 10:58

The last time I tried it didn't =/

Max Weninger commented on 2007-03-12 11:41

The scrolling "type" depends on the length of the string

Max Weninger commented on 2007-03-12 11:43

But can be influenced from the setting "Bidirectional Scroll Limit"
(which is 50% by default IMHO)

Marc R. commented on 2007-03-12 12:02
The scrolling "type" depends on the length of the string

I just found that one out.
so it's a feature :)

Dave Hooper commented on 2007-03-12 20:48

Dano: I'll take a look at updating in a minute or two, I confess I didn't try all targets

Dave Hooper commented on 2007-03-12 21:06

Dano: hm, the H3xx lcd driver seems to build ok here. Did you get a fresh update of source from svn and did you apply any other patches first (and/or were any conflicts or failures reported when you applied the scroll_margins_20070310 patch?)

Marc R. commented on 2007-03-14 09:55

Is the code for the file sound_menu.c needed?
This file has been removed from the current svn-tree.

Dan Train commented on 2007-03-14 10:15

Yeah I just noticed that when I went to try and confirm stripwax's questions with the latest build.

Anyway previously I had got the 20070312 source from the archive daily builds page, and I had also applied the latest album art patch and eq wps tags patch, and there weren't any errors or conflicts applying any of the patches.

Thanks very much for your help.

Marc R. commented on 2007-03-16 04:45

Will there be a patch for the current svn-build?

Dave Hooper commented on 2007-03-16 23:48

Rasmusma: Sync'd to svn as-of a couple of hours ago
Dano: hopefully this time the patch will work for you also, I've manually updated the 1bit remote driver

Dan Train commented on 2007-03-18 18:29

It works beautifully now, thank you very much for your help stripwax!

David Hall commented on 2007-03-28 00:40

I am not 100% confident in my sync to 12944. Someone probably should test with a LCD remote. (Or just manually sync stripwax's lastest work.)
It appears to work fine on my iPods. (Famous last words.)

Chris Olin commented on 2007-04-05 04:21

With some minor failed hunks in gwps/gui/gwps-common.c that can easily be manually added in by hand, Soap's patch works the best right now.

Marc R. commented on 2007-04-05 04:31

Would there be any problems to add this to the svn-tree?

Jordan commented on 2007-04-05 21:25

I can't seem to get this to work in the new SVN.
I get "gui/gwps-common.c: In function ‘get_line’:
gui/gwps-common.c:1421: error: ‘fmt’ undeclared (first use in this function)
gui/gwps-common.c:1421: error: (Each undeclared identifier is reported only once
gui/gwps-common.c:1421: error: for each function it appears in.)"
And 1421 is the whole "case 'm': /* margins */" hunk of code.
Can someone upload a sync'd patch, or at least tell me what I've done wrong?

Nicolas Pennequin commented on 2007-04-06 01:09

Some parts of the patch will need to be rewritten to be made compatible with the WPS tokenizer.

Dave Hooper commented on 2007-04-07 12:57

I'm working on the rewrite.

Dave Hooper commented on 2007-04-07 16:14

Sync'd. Added new WPS virtual tokens to handle the left and right margins. Shout if you find any problems with scrolling margins in WPSes that used to work but break with this new patch.

Dave Hooper commented on 2007-04-08 09:54

Sync'd again.

Dave Hooper commented on 2007-04-09 10:05

Sync'd again. Unfortunately the new WPS parser is a moving target, and the underlying function definitions keep changing..

Dave Hooper commented on 2007-04-12 21:24

Sync'd again. Also fixes problem of dynamic data on non-scrolling lines not updating correctly now that gwps-common has been fixed in svn to handle WPS_REFRESH_SCROLL properly (the %m tag should NOT have had WPS_REFRESH_SCROLL flag in wps_parser.c in the first place - apologies)

Oleg G commented on 2007-04-15 22:06

Needs another sync. Applies with many failed hunks.

Travis Tooke commented on 2007-04-16 03:59

I think you may be doing something wrong. I just updated to revision 13174 just now and this patch applied without incident.

Oleg G commented on 2007-04-16 11:31

Hmm. I'll try to apply to a clean SVN then. Maybe some other patch is messing it up.

Dave Hooper commented on 2007-04-24 12:59

Sync'd and fixed. Today's wps parser fix in svn revealed an off-by-one error in my parse function which was now caught by the improved error check (meaning wps that used scroll margins were no longer loadable!) Also needed resyn due to bookmark.c change in svn

Marc R. commented on 2007-04-24 14:58

hmm
I ask again: Would there be any problems to ADD this patch to the svn tree?

Gary Light commented on 2007-04-25 01:24

for anyone building for a gigabeat and getting errors on debug_menu.c
edit line 1140 in apps/debug_menu.c so it reads:

lcd_setmargins(0, LCD_WIDTH, 0);

Dave Hooper commented on 2007-04-26 21:33

sync'd and updated. If using albumart too, use latest albumart patch too to avoid sync conflicts.

Chris Banes commented on 2007-04-28 22:17

Has anyone tried this? Because I'm having trouble loading up WPS's. It's either this or the Album Art patch.

Dave Hooper commented on 2007-04-29 19:21

Senab - I have been running my own build using this patch and the album art patch without any problems.

Jon commented on 2007-05-10 02:42

Out of sync- I fixed one simple update hunk but someone who knows the code needs to look at the others and see if they're needed

Hunk 1 of apps/onplay.c and Hunk 20 of apps/debug_menu.c

Attached is the otherwise synced version.

Travis Tooke commented on 2007-05-15 06:48

I wish I could say whether or not those 2 hunks are truly necessary, but for what it's worth I haven't had any problems whatsoever using the broken patch.

Jon commented on 2007-05-15 13:15

Same here.

Travis Tooke commented on 2007-05-28 04:33

FS #7158 being commited breaks this patch.

Gary Light commented on 2007-05-28 14:36

Are you sure it's broken? I took a look at the .rej and it's trying to add rb→lcdsetmargins(0,LCD_WIDTH, 0) so I took a look at bookmark.c for
rb→lcdsetmargins, this function (or whatever its called) isn't in bookmark.c anymore, so I tried compiling and it works. Did you try compiling?

Travis Tooke commented on 2007-05-29 06:34

Ah, I should probably do my homework before posting.

Matthew Schneider commented on 2007-06-19 22:20

I think it's out of sync

Tan Yu Sheng commented on 2007-06-20 12:33

I think this works.

Matthew Evans commented on 2007-06-22 18:45

I get two hunks failing and it won't make.

Andrej W. commented on 2007-06-23 02:22

i too get a failure at lcd.h (hunk #2 at 393) and at lcd-16bit.c (hunk #7 at 947)

what to do…

Tan Yu Sheng commented on 2007-06-23 19:39

Sync again then.

Jake Melvin commented on 2007-07-01 09:12

While the patch is working for me, I seem to be having trouble using the regular %m tag, such that it is a normal line. No matter what I do, the margin designated with just %m retains the same left position as the custom lines. Is anyone else having this issue, or perhaps I'm doing something wrong?

Travis Tooke commented on 2007-07-02 02:24

To clarify, when you say custom lines are those custom lines as in jbuild %e.. lines? If so I think that's the problem, mixing %e with %m on the same wps. If that's not the case then I have no idea.

Jake Melvin commented on 2007-07-02 07:04

I am actually not making use of the %e tag, but I found a solution to my problem anyways. The solution was simply to use a %m tag along with |x1|x2| positions related to the width of my wps, which was |0|240|. This does make sense, although it isn't what I had expected.

Matthew Schneider commented on 2007-07-02 17:55

That's because that is how the scrolling margins patch works. Every text line needs to have a margin applied.

Dieter commented on 2007-07-06 14:47

Out of sync.

Nick Brackley commented on 2007-07-11 11:11

Sync Please :)

Chris Banes commented on 2007-07-12 15:06

Resync :)

It's just the older version with the hunks that didn't patch taken out (they weren't needed anymore).

Phil Light commented on 2007-07-28 11:32

Changed to apply cleanly following this morning's SVN changes.

Mike Kasberg commented on 2007-08-16 04:02

My first re-sync :)

Only one small change; it should work with SVN as of 2007-08-15.

Dieter commented on 2007-08-18 20:29

The last patch does not work. It misses the changes in alarm_menu.c (which were included in the previous patch).

Jake Melvin commented on 2007-08-21 00:04

That's odd. It applies just fine for me on a Gigabeat F40.

Mike Kasberg commented on 2007-08-30 01:45

It builds fine, but I think it might be causing some weird behavior. Perhaps someone should do a proper re-sync based on the previous patch.

Dieter commented on 2007-09-02 08:27

When trying to compile for the iPod Video with 64MB this patch still does results in the following compiling error:

… CC alarm_menu.c
alarm_menu.c: In function 'alarm_screen':
alarm_menu.c:73: error: too few arguments to function 'screens[i].setmargins'
make[1]: * [/home/user/rockbox/build/apps/alarm_menu.o] Error 1
make:
* [build] Error 2

When including the previously included changes on alarm_menu.c it compiles fine.

Travis Tooke commented on 2007-09-14 03:10
Akio Idehara commented on 2007-09-14 15:14

resync and fix buggy jumping strings

Travis Tooke commented on 2007-09-14 17:37

Hmm.. out of curiosity what exactly was it doing and what lines did you change? I pretty much just did a resync of 2007-08-15 with the hunk for alarm_menu.c put back in. I never noticed it doing anything on my iPod and I tested it on various themes some plain, some customline.

Akio Idehara commented on 2007-09-14 22:39

I added setmargins(0, LCD_WIDTH, 0) in get_line().

Please test follow wps with 2 patches.
— %wd
%s%m|25|100|%?it<%it |%fn>
%ar%pc/%pt
%arTrack %?in<%in|unknown> (%pp/%pe)
%s%m|100|175|%?it<%it |%fn>

Akio Idehara commented on 2007-09-14 23:38

I change from LCD_WIDTH to gwps→display→width for rwps.

Travis Tooke commented on 2007-09-15 05:45

I can definitely see that bug now with that test wps! Thanks for the fix!

Anonymous Submitter commented on 2007-09-24 14:07
Chris Banes commented on 2007-09-27 19:01
Akio Idehara commented on 2007-09-29 04:02

A long string was displayed protruding to the right when new track info was loaded.
So I fix it.

Akio Idehara commented on 2007-10-02 00:48

more suitable for rwps.

Martin Buck commented on 2007-10-04 12:24

When setting margins for a non-scrolling line (i.e. just %m without %s), still the complete line is cleared by write_line()/gui_wps_refresh(), destroying e.g. album art that might be next to the line and which is supposed to be protected by the margins.

The attached patch fixes this for me. Does anybody see a reason why it shouldn't be merged into the scroll-margins patch?

Akio Idehara commented on 2007-10-05 16:33

add lcd-charcell.c bogus support.
Now all target can be supposed to use this patch…

Travis Tooke commented on 2007-10-06 00:01

@Martin, I agree that should be merged too, and on another note thanks for posting that. Looking at that area of code made me rediscover something I meant to do a long time ago on another patch and just forgot to.

Travis Tooke commented on 2007-10-06 00:10

Sorry for the double post but this just occurred to me, I'd still go with display→width on that instead of scroll_width.

Akio Idehara commented on 2007-10-06 01:26

Does this patch fix that problem?

Akio Idehara commented on 2007-10-06 03:41

Fix clearing the center aligned string.
Sorry for above buggy patch…

Travis Tooke commented on 2007-10-07 10:58

@idak, what do you think of something like:
if (left_width || center_width || right_width) {

             display->set_drawmode(DRMODE_SOLID|DRMODE_INVERSEVID);
             display->fillrect(left_xpos, ypos, display->width, string_height);
             display->set_drawmode(DRMODE_SOLID);
         }
Akio Idehara commented on 2007-10-07 11:27

@tdtooke, Can I have your test wps, please?
I don't understand what has occurred…

Travis Tooke commented on 2007-10-07 17:18

Nothing happened, the patch performs flawlessly. I just thought that would be easier. Well….actually my motivation is a little bit selfish. If that part looked that way then all I'd have to do to sync another patch would be to just manually edit the patch, otherwise I have to do a lot of typing.;-)

Jacob Brooks commented on 2007-10-18 23:36

Out of sync, I tried to figure out the problem but it looks like a code change and I have no experience coding.

Akio Idehara commented on 2007-10-19 08:07
Jacob Brooks commented on 2007-10-21 20:37

getting a error on main_menu.c, I looked for the proper line but there must have been another code change.

Thomas Martitz commented on 2007-10-21 20:49

I just igonired that HUNK, with success :)

Jacob Brooks commented on 2007-10-21 23:29

it compiled but does it work?

Thomas Martitz commented on 2007-10-21 23:41

If you mean, that all my themes are working as they should, yes! ;)

Travis Tooke commented on 2007-10-21 23:48

You don't' have to worry about it. They're doing things a different way now on main_menu.c that doesn't need that. Now if they had just moved that you'd not just get a warning, it wouldn't compile at all since you'd be calling setmargins with too few parameters.

Nick Brackley commented on 2007-10-24 01:48

Along with main_menu.c failing, debug_menu.c and scroll_engine.h now fail. Could someone please resync?

Akio Idehara commented on 2007-10-24 10:40
Jacob Brooks commented on 2007-11-12 23:15

Sync errors on todays SVN

8 out of 14 hunks FAILED – saving rejects to file apps/gui/gwps-common.c.rej
1 out of 1 hunk FAILED – saving rejects to file apps/gui/gwps.h.rej
1 out of 3 hunks FAILED – saving rejects to file apps/gui/wps_parser.c.rej
1 out of 16 hunks FAILED – saving rejects to file apps/debug_menu.c.rej

Nicolas Pennequin commented on 2007-11-12 23:19

that's because part of the patch was committed. Should be an easy sync.

Jacob Brooks commented on 2007-11-12 23:25

Yeah I saw that but why only left margins why just commit this patch?

Nicolas Pennequin commented on 2007-11-12 23:31

Adding support for the right margin is the intrusive part of this patch. The left margin part is pretty trivial in comparison and only affect the WPS code. More than what has been committed isn't wanted because viewports are planned.

Travis Tooke commented on 2007-11-13 01:03

With a bit of a rewrite this could be made to assume that no declaration of margins means %m|0|320| and %s|leftmargin| means %s%m|leftmargin|320|. This might be the best compromise with the latest commit. Thoughts?

Travis Tooke commented on 2007-11-13 01:07

or should I say where I listed what it'd be on an iPod 5.5 it should be %m|0|LCD_WIDTH| or something like that

Travis Tooke commented on 2007-11-13 04:22

Very quick and dirty. With this if you're using a new WPS which might make use of %s|leftmargin| that will work and as before it works with %m|leftmargin|rightmargin|. As always if you set a margin for one line you set a margin for all lines (with %m that is). I have noticed that placement of the %s tag in the middle somewhere as opposed to the beginning of the line can cause problems so if you're using partial-scroll be aware of that. Also if you use the scrollwheel acceleration patch for the iPod that patch will add a setmargins call that you will have to manually fix. I didn't include that because that would introduce a dependency.

Travis Tooke commented on 2007-11-14 10:36

Another quick and dirty. This one basically just extends the %m functionality just committed to include the rightmargin. As in SVN it will work with just %m or %m|leftmargin| but will also work with %m|leftmargin|rightmargin| as before. I changed the name because it really isn't scroll-margins anymore. Maybe this should have a separate page on flyspray? Also as before of you use the iPod scrollwheel acceleration patch there is a setmargins call in debug_menu.c that you need to manually set. I would have included that but that would introduce a dependency.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing