Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#4762 closed Bug (fixed)

Buggy scrollbar in source view IE8

Reported by: Kingsquare Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.2
Component: UI : Toolbar Version: SVN (CKEditor) - OLD
Keywords: Confirmed IE Review+ Cc: ckeditor@…, kevin@…

Description

This is the weirdest bug i've seen in a while:

In an editor instance in Internet Explorer 8, with compatibility mode disabled, when looking at 'code view' and with some :hover declaration declaren anyware in the page (that contains the editor), the scrollbar of the editor starts 'blinking' and scrolling up when hovering over and out the textarea.

The :hover declaration is the containing page could be something like

<style type="text/css">
   somethingthatisnotA:hover { color: #ABC; }
</style>

and that declaration has nothing to do with the editor at all! To reproduce this problem, please try the following

Attachments (2)

4762.patch (2.1 KB) - added by Garry Yao 8 years ago.
4762_2.patch (3.1 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Confirmed IE added
Milestone: CKEditor 3.2

IE is always so lovable... that's really weird!

I'm targeting to the 3.2 for investigation. I hope you've been able to workaround it for your needs.

comment:2 Changed 8 years ago by Garry Yao

Unable to reproduce on trunk.

comment:3 Changed 8 years ago by Kingsquare

Cc: ckeditor@… added

The testpage set up @ http://reggino.kingsquare.nl/bugreport/ckeditor/test.html contains last nightly build / current trunk version(rev 4658) and can be used to reproduce the problem.

comment:4 Changed 8 years ago by Frederico Caldeira Knabben

I was definitely able to reproduce the problem by using the provided test page URL.

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

#4811 has been marked as DUP. A sample file and instructions are available there.

comment:6 Changed 8 years ago by Iuliu Sandu

I experienced the same problem in version 3.1 of ckeditor. I added the following code at line 72 (just above the line that says: styles.height = holderElement.$.clientHeight + 'px';):

styles.width = holderElement.$.clientWidth + 'px';

Of course this has the knock-on effect of putting a fixed width on the textarea but it does the job. A bit more about this IE8 bug here http://grantovich.net/posts/2009/06/that-weird-ie8-textarea-bug/

Changed 8 years ago by Garry Yao

Attachment: 4762.patch added

comment:7 Changed 8 years ago by Garry Yao

Component: GeneralUI : Toolbar
Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned
Version: SVN (CKEditor)

Proposing a workaround based on iuliusandu's idea.

comment:8 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review- added; Review? removed

This bug existed previously on that part of code but it affected only !CKEDITOR.env.ie8Compat, the listener for afterCommandExec isn't removed in the unload so collapsing the toolbar afterwards generates an error.

If the window is resized horizontally (with the editor in source mode and in its default state: not maximized, not resized), then the textarea doesn't resize. It should be possible to add a listener to the resize of the window to catch this situation (and as in the previous case, it must be removed at unload)

Changed 8 years ago by Garry Yao

Attachment: 4762_2.patch added

comment:9 Changed 8 years ago by Garry Yao

Keywords: Review? added; Review- removed

Providing a patch targeting alfonsoml's review points.

comment:10 Changed 8 years ago by Kevin Williams

Cc: kevin@… added

comment:11 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review+ added; Review? removed

comment:12 Changed 8 years ago by Kevin Williams

garry.yao's second patch for this defect appears to resolve defects #4761, #4775, #4718, #5086, and of course #4762 in my local testing; it applies seamlessly for me to the release tagged 3.1. Thank you garry!

comment:13 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5008].

comment:14 Changed 7 years ago by Garry Yao

#5106 and #3817 has been marked as duplicates.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy