Focus jump to top when reentering the iframe (in IE8)

[Ulrich]

Steps to reproduce :

• Open demo in IE8
• Delete some text, so that the body is smaller than the iframe
• Place Focus in the text somewhere
• Click outside of the editor
• Click in the white space in the iframe below the body

Focus jumps to position 1, but should be at the end

Browser name and OS : IE8 on Windows XP

[Jakub Ś]

This is rather browser dependable. If we can't do anything about it please close the issue.

Opera remembers the position.

IE places cursor at the beginning.

FF and chrome place cursor at the end.

[Jakub Ś]

If we can't do anything about it please close the issue.

@Ulrich I'm not sure if you have closed the issue because of above comment or because you have deducted that this is a won't fix.

If because of my comment, than I must say sorry for confusing you. I have written that to our senior developers:)

[Garry Yao]

I've just found a much unobtrusive way to fix #1659, also find the focus grabber stuff is not anymore necessary for other browsers as well, which is introduced back in #5622.

[Garry Yao]

#7326 and #8074 are related.

[Sa'ar Zac Elias]

IE7 is still affected.

[Garry Yao]

New patch takes care of IE6/7, even not having the dragging selection feature in IE8/9/Quirks , it eliminates the selection lost issue.

Due to an Trac error I can't obsolete the patches, 7932_3.2 is the latest.

[Lucio Fassio]

Garry,

the patches from 7932_2.patch to 7932_3.2.patch cannot be downloaded for an internal error in Trac; can you publish it somewhere else?

Thanks

[Garry Yao]

Replying to lfassio:

Garry,

the patches from 7932_2.patch to 7932_3.2.patch cannot be downloaded for an internal error in Trac; can you publish it somewhere else?

Thanks

It shows me that it's the diff that triggers the Trac error with the patch, 7932_3 is the latest one.

[Frederico Caldeira Knabben]

I'm not able to apply this patch (malformed). Please try creating it again.

[Frederico Caldeira Knabben]

Removing stable code is a very dangerous things. To support it, we need to have tests in place for related tickets: #1659, #5622, #5634 and #5684 (at least). Only after that we'll be able to consider this patch.

[Wiktor Walc]

Just a note that it might be worth to check #5538 according to comment added by Garry.

[Jakub Ś]

Oyher related tickets (duplicates) #6359, #6013, #5538.

[Jakub Ś]

Issues that seem to be fixed by last patch from Garry: #6013, #7932, #5538, #6359, #8658, #8643, #8729, #8074, #7326, #8455

[Frederico Caldeira Knabben]

​http://ckeditor.t/tt/7932/1.html

[Frederico Caldeira Knabben]

I've created tests for several related tickets and everything is passing, which indicates that we're almost there.

The test for this ticket is not passing on IE9+Compat, IE8+Compat and IE6 (and probably on IE7 and IE+Quirks). It passes on IE9, IE8 and other browsers.

[Ruud Vossen]

(sorry for the comment. I just want to add myself to cc, but if i leave the comment field blank the content is 'potential spam'. Maybe I do something wrong?)

[Frederico Caldeira Knabben]

Replying to ruud_bmg:

(sorry for the comment. I just want to add myself to cc, but if i leave the comment field blank the content is 'potential spam'. Maybe I do something wrong?)

(That should be ok, sorry... we'll be investigating.)

[Matthew Leffler]

Adding to CC...

[Garry Yao]

I didn't mean to address the ticket in IE6/7...while, I agree... we can be a bit better here, so new patch should be enough to align this behavior with IE9.

[Garry Yao]

Fixed with [7374].

[Wiktor Walc]

~@ruud_bmg - Akismet was rejecting your request to add you to the CC field because it looks like you have set at that time more than one email address in your account settings (the one that is assigned now to the user "gogobu").~

@ruud_bmg - thanks for reporting the problem. Indeed "Akismet says content is spam" when cc-ing as a second person, without posting anything.

[Ruud Vossen]

@wwalc: ok, is there anything i can do to fix/prevent that? I don't know the user gogobu, so I don't know how the e-mail address got in my account settings.

[Wiktor Walc]

@ruud_bmg - sorry for the confusion, I just noticed in logs that you have tried to set the CC field to two email adresses, but in fact it was because gogobu cc-ed himself two weeks earlier. It's a system error, I've marked your earlier changes that failed as "ham", let' s see if it will help.