Description

If you have a page that is bigger than your screen size (in this example, a really large table) and you want to scroll to the right, Internet Explorer 7 forces you to scroll back to the left.

• IE8 running in IE7 compatibility mode has the same problem.

Steps to reproduce

1. See a really long table like the below.
2. In Internet Explorer 7, try and scroll to the right.
3. Become frustrated.

Example

||<tablewidth="200px">sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf||sadgvsfdbdgdfdabf|| || || || || || || || || ||
|| || || || || || || || || || || || || || || || || || || || || || || || || || || || || ||
|| || || || || || || || || || || || || || || || || || || || || || || || || || || || || ||

Component selection

• This is a problem with MoinMoin/web/static/htdocs/common/js/common.js

Details

Workaround

I don't use double click editing. I have commented out line 801 and things still seem to work ok for me in IE & Firefox. Things like the Edit(GUI) still work etc. I don't pretend to understand how it all works. Its probably a silly thing to do although I hope its only breaking *really* old browsers.

• // ensure init will be run by obsolete browsers
  addLoadEvent(runScrollTextareaInitOnce);

Joseph, please test the following and report back. Edit common.js, delete everything beginning with the line // speed up most browsers -- run my function As Soon As Possible to the end and replace it with the following:

  snip: obsolete, see below

• -- RogerHaase 2010-07-08 19:41:01

Yes that looks good to me. I can't immediately find anything broken either. I don't use things like double click editing though, which I believe is around that point in the code?

• -- JosephPrice 2010-07-09 08:07:09

Here is a revised fix that simplifies the initialization startup. 1.9.3 users can replace /htdocs/common/js/common.js with the file below:

common.js

Here is a mercurial changeset of the above against 1.9.3 tip as of 2010-07-23:

autoscroll.diff applied by http://hg.moinmo.in/moin/1.9/rev/7a83cc907f68

• -- RogerHaase 2010-07-23 17:51:15

  Yep, looks like that does the job and I can't see anything broken myself. -- JosephPrice 2010-08-05 08:29:27

Discussion

Plan

• Priority:
• Assigned to:
• Status: patch was applied

CategoryMoinMoinBugFixed