CKEditor doesn't scroll to show matched words when searching or replacing

[Jakub Ś]

1. Paste the following code

   <ul>
   	<li>
   		Copy form word</li>
   	<li>
   		copy from word 3 level lists</li>
   	<li>
   		klawiszeTemty kt&oacute;re powtarzają się i warto najpierw zrobić search.</li>
   </ul>
   <p>
   	<strong>Temty kt&oacute;re powtarzają się i warto najpierw zrobić search.</strong></p>
   <p>
   	&nbsp;</p>
   <ul style="list-style-type: decimal;">
   	<li>
   		Copy form word</li>
   	<li>
   		copy from word 3 level lists</li>
   	<li>
   		klawisze
   		<div>
   			<strong>Temty kt&oacute;re powtarzają się i warto najpierw zrobić search.</strong></div>
   		<div>
   			&nbsp;</div>
   	</li>
   	<li>
   		Copy form word</li>
   	<li>
   		copy from word 3 level lists</li>
   	<li>
   		klawiszeTemty kt&oacute;re powtarzają się i warto najpierw zrobić search.<br />
   		<br />
   		&bull; Copy form word<br />
   		&bull; copy from word 3 level lists<br />
   		&bull; klawisze<br />
   		<p>
   			&nbsp;</p>
   	</li>
   </ul>
   

2. Go to wysiwyg mode.
3. Search for word "copy"
4. You will se that editor doesn't scroll to show matching words.

This dopesn't work under FF3.6 in WIN7 and XP since CKEditor version 3.0.1

In CKEditor version 3.0 "cycle search" doesn't work at all. When editor reaches the end of the document it doesn't come back up and displays the dialog - no occurrences found.

NOW THE FUN PART: The text I have pasted is big enough for scrolling, but try to double the text by pasting it once more. Now everything works. Magic :)

NOTE : I HAVEN'T TESTED IT ON FF4 YET.

[Jakub Ś]

The issue concerns all Browsers

[Jakub Ś]

This is just a wild guess, but maybe toolbar height gets somehow included in the editing area and that is why sample text with particular height is not scrolled properly.

[Garry Yao]

Propose a rewrite of element::scrollIntoView, deprecate "alignTop" param as it's not actually in effect.

[Jakub Ś]

An extension to this ticked based on forum post ​http://cksource.com/forums/viewtopic.php?f=11&t=22899&p=58594#p58594. This part is reproducible in all versions of IE from CKE 3.0.2.

1. Paste the following code or use the one attached to original ticked:

   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   <p>
   	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
   

2. Put the cursor in the first line
3. PageDown all the way
4. Now try to PageUp all the way.

Result: Cursor will move to the first line but CKE won't scroll up just like in case of find/replace.

From CKE 3.0 to CKE 3.0.1 editor didn't scroll both ways. From CKE 3.0.2 it doesn't scroll up.

[Teresa Monahan]

The problem here is the following code:

// offset value might be out of range(nagative), fix it(#3692).
offset = offset < 0 ? 0 : offset;

// Scroll the window to the desired position, if not already visible(#3795).
var currentScroll = win.getScrollPosition().y;
if ( offset > currentScroll || offset < currentScroll - winHeight )
	win.$.scrollTo( 0, offset );

The offset is reset to zero if it is a negative value BEFORE the conditional that determines whether the offset is in the current viewport or not. It should not be reset until AFTER this check has occurred.

The original fix for #3692 did this correctly, so I'm not sure why or when this got changed. In any case changing the code to the following resolves the issue:

// Scroll the window to the desired position, if not already visible(#3795).
var currentScroll = win.getScrollPosition().y;
if ( offset > currentScroll || offset < currentScroll - winHeight )
	win.$.scrollTo( 0, offset > 0 ? offset : 0  );		// offset value might be out of range(nagative), fix it(#3692).

This is a high priority issue for us. Can it be fixed in the 3.6.3 release please?

[Garry Yao]

Cancel the review as of comment:4:ticket:7946.

[Jakub Ś]

#8625 was marked as duplicate

Comment from this ticket looks similar to comment:10

To reproduce:

1. Go to replacebycode sample
2. Press Ctrl+A , and Delete (clear the contents)
3. Type 'test'
4. Put the cursor at the beginning of the word ^test
5. Start pressing enter, until scrollbar appears. (It should be about 8 Enter key presses)
6. Press pageUp

Result: Cursor jumps up but the editor does not scroll to show it.

Issue has been reproducible in all IE's from CKEditor 3.0

NOTE:If you press Enter key many times (let's say about 20 or 30) pageUp will work

[Garry Yao]

Ticket is fixed by #7946, tc from comment 10 is moved #8821.