Opened 14 years ago

Closed 14 years ago

Last modified 14 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 14 years ago.
4762_2.patch (3.1 KB) - added by Garry Yao 14 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 14 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 14 years ago by Garry Yao

Unable to reproduce on trunk.

comment:3 Changed 14 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 14 years ago by Frederico Caldeira Knabben

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

comment:5 Changed 14 years ago by Frederico Caldeira Knabben

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

comment:6 Changed 14 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 14 years ago by Garry Yao

Attachment: 4762.patch added

comment:7 Changed 14 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 14 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 14 years ago by Garry Yao

Attachment: 4762_2.patch added

comment:9 Changed 14 years ago by Garry Yao

Keywords: Review? added; Review- removed

Providing a patch targeting alfonsoml's review points.

comment:10 Changed 14 years ago by Kevin Williams

Cc: kevin@… added

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

Keywords: Review+ added; Review? removed

comment:12 Changed 14 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 14 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5008].

comment:14 Changed 14 years ago by Garry Yao

#5106 and #3817 has been marked as duplicates.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy