The placement of the cursor sometimes was wrong under Windows. (The bug
could be reproduced under Linux by changing the configuration files.)
The problem was caused by the height of the text box and the permission
to wrap the text on a new line. This caused the cursor position to
return to the beginning of the line.
Fixes bug #17220 using patch #3822.
Most references to and dependencies on Subversion have been removed.
"+svn" is now "+dev". Files that can't be fixed yet have a FIXME-GIT
comment in them; most of these are in the website tests.
------------------------------------------------------------------------
Tag-Name: v0_4_8rc1
This commit was manufactured by cvs2svn to create tag 'v0_4_8rc1'.
This purportedly fixes bug #18793.
(Candidate for 1.10, PLEASE TEST!)
The conditionals in question:
> if (scroll_type == ONSCREEN || ONSCREEN_WARP) {
This *always* evaluates to true when scroll_type != ONSCREEN, because
ONSCREEN_WARP is an enum member with a non-zero value. The bogus code is
effectively the same as:
> if ((scroll_type == ONSCREEN) || true) {
Which was most likely not the author's intention (fendrin in r49988).
Therefore, I've replaced the conditionals with what should be the
correct way to do what I suspect was meant above:
> if (scroll_type == ONSCREEN || scroll_type == ONSCREEN_WARP) {
This seems to fix the viewport centering issues for me, but it would be
nice if people reported their own experience with the fix before
backporting it to 1.10.
This is possible as a side-effect of 2013-02-03T19:05:22Z!jt_coding@verizon.net. I think it looks
kind of cool, but if it turns out to be a bad idea, it can be easily
reverted (just a matter of changing a single "false" to "true").
there was a chance of them not firing ever since WML division was
changed to floating point.
Also changelog entries for this and 2013-02-01T04:48:38Z!jt_coding@verizon.net.
...fields and implement a button to toggle them all at once
Also needed to do some refactoring to achieve the tracking part. Not
that anyone really needs to see that in a separate commit.
