Opened 6 years ago

Closed 6 years ago

Last modified 6 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 6 years ago.
4762_2.patch (3.1 KB) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 6 years ago by fredck

  • Keywords Confirmed IE added
  • Milestone set to 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 6 years ago by garry.yao

Unable to reproduce on trunk.

comment:3 Changed 6 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 6 years ago by fredck

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

comment:5 Changed 6 years ago by fredck

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

comment:6 Changed 6 years ago by iuliusandu

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 6 years ago by garry.yao

comment:7 Changed 6 years ago by garry.yao

  • Component changed from General to UI : Toolbar
  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned
  • Version set to SVN (CKEditor)

Proposing a workaround based on iuliusandu's idea.

comment:8 Changed 6 years ago by alfonsoml

  • 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 6 years ago by garry.yao

comment:9 Changed 6 years ago by garry.yao

  • Keywords Review? added; Review- removed

Providing a patch targeting alfonsoml's review points.

comment:10 Changed 6 years ago by kevinw

  • Cc kevin@… added

comment:11 Changed 6 years ago by alfonsoml

  • Keywords Review+ added; Review? removed

comment:12 Changed 6 years ago by kevinw

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 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [5008].

comment:14 Changed 6 years ago by garry.yao

#5106 and #3817 has been marked as duplicates.

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