426698 - Hitting 'enter' in designMode causes the outer page to scroll

Status: RESOLVED INCOMPLETE

(Tech Evangelism Graveyard :: English US, defect)

Description

[voracity]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5) Gecko/2008032620 Firefox/3.0b5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5) Gecko/2008032620 Firefox/3.0b5

Visit the example URL, resize the vertical height of the window until you get a scrollbar, then scroll the page down to the bottom. Type something into the rich text editor and the hit 'enter'. The entire page will scroll back up.

Reproducible: Always

Comment 1

[Ria Klaassen (not reading all bugmail)]

Regression window is
http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=1199708400&maxdate=1199713799
Seems caused by Bug 332246.

Comment 2

[Martijn Wargers (dead)]

I've investigated this:
In tiny_mce.js, there is the insertPara function there is the (only one) mention of scrollIntoView(0).

That's exactly what has changed in bug 332246, scrollIntoView(0) (or false) has been changed to do what IE is doing also.

Now, IE doesn't show the problem on that site, because it's not following that code path:
if(!isIE&&s.force_p_newlines){ed.onKeyPress.add(function(ed,e){if(e.keyCode==13&&!e.shiftKey){if(!t.insertPara(e))Event.cancel(e);}});

Tinmymce should change their code to scrollIntoView(1), that would give the desired behavior.

Comment 3

[Martijn Wargers (dead)]

I've filed a bug in sourceforge for it now:
http://sourceforge.net/tracker/index.php?func=detail&aid=1933349&group_id=103281&atid=635682

Comment 4

[Martijn Wargers (dead)]

This is apparently solved in TinyMCE by completely removing scrollIntoView (!).
So I guess it's just a matter of waiting before the examples get updated.

Comment 5

[voracity]

FCKEditor has the same problem:

http://www.fckeditor.net/demo

I've filed a ticket:

http://dev.fckeditor.net/ticket/2086

I'm not sure if other designMode editors would also have problems?

Comment 6

[Martijn Wargers (dead)]

Yeah, it seems very likely that FCKEditor has the same problem. I also saw scrollIntoView(false) being used in fckeditorcode_gecko.js.

Comment 7

[Chris Lawson (gone)]

INCOMPLETE due to lack of activity since the end of 2009.

If someone is willing to investigate the issues raised in this bug to determine whether they still exist, *and* work with the site in question to fix any existing issues, please feel free to re-open and assign to yourself.

Sorry for the bugspam; filter on "NO MORE PRE-2010 TE BUGS" to remove.