Opened 6 years ago

Closed 6 years ago

#6146 closed Bug (fixed)

IE: Floating panels offset when editor is within a container that has defined width

Reported by: damo Owned by: fredck
Priority: Normal Milestone: CKEditor 3.4.1
Component: UI : Floating Panel Version: 3.0
Keywords: IBM IE Cc: satya_minnekanti@…

Description

See attached screenshot for example.

Steps to reproduce:

  1. Open sample provided with ticket in IE7.
  2. Ensure that the window size of the browser is small enough to cause a horizontal scrollbar to appear in the window
  3. Open a panel.

Result: the floating panels render off-screen. This is a major globalization problem.

Sample to test with:

<html>
	<head>
		<meta content="text/html; charset=utf-8" http-equiv="content-type" />
		<script type="text/javascript" src="ckeditor_source.js"></script>
		<script type="text/javascript">
			function loadEditor() {
				CKEDITOR.appendTo("content", {language: "he", skin: "kama"});
			}
		</script>
	</head>
	<body onload="loadEditor();" dir="rtl">
		<div style="width: 800px; border: 1px dashed black;">
			<div id="content">
		</div>
	</body>
</html>

Attachments (5)

IE_FloatingPanelProblem.png (13.2 KB) - added by damo 6 years ago.
6146.patch (3.2 KB) - added by Saare 6 years ago.
IE_FloatingPanelProblem_2.png (10.6 KB) - added by damo 6 years ago.
IE_FloatingPanelProblem_3.png (11.6 KB) - added by damo 6 years ago.
6146_2.patch (1.3 KB) - added by fredck 6 years ago.

Download all attachments as: .zip

Change History (17)

Changed 6 years ago by damo

comment:1 Changed 6 years ago by Saare

  • Status changed from new to confirmed

comment:2 Changed 6 years ago by Saare

  • Owner set to Saare
  • Status changed from confirmed to assigned

Changed 6 years ago by Saare

comment:3 Changed 6 years ago by Saare

  • Milestone set to CKEditor 3.4.1
  • Status changed from assigned to review
  • Version set to 3.0

comment:4 Changed 6 years ago by damo

This definitely works better now.

When the user scrolls to the left, it still throws the panel alignment off a bit (see attachment IE_FloatingPanelProblem_2.png.

Also, this issue affects the colour picker panels too, see IE_FloatingPanelProblem_3.png.

Changed 6 years ago by damo

Changed 6 years ago by damo

comment:5 Changed 6 years ago by Saare

  • Status changed from review to review_failed

comment:6 Changed 6 years ago by Saare

  • Keywords IE added

Weird, but I'm not able to reproduce it anymore with IE8 in all modes in both trunk and the 3.4 branch.

comment:7 Changed 6 years ago by satya

  • Cc satya_minnekanti@… added

comment:8 Changed 6 years ago by fredck

  • Owner changed from Saare to fredck
  • Status changed from review_failed to assigned

I'm able to reproduce the problem with IE8 in all modes.

comment:9 Changed 6 years ago by fredck

Actually, I've noticed that the TC is on quirks, so I was able to reproduce it on IE8+Quirks and IE8+Compat. It's ok with IE8+Standards.

Changed 6 years ago by fredck

comment:10 Changed 6 years ago by fredck

  • Status changed from assigned to review

I've noticed that even IE8 mode has issues in this tc, even if in a different way.

The provided patch should be able to fix it in quirks (IE6), IE7 and IE8 modes.

comment:11 Changed 6 years ago by Saare

  • Status changed from review to review_passed

comment:12 Changed 6 years ago by fredck

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

Fixed with [5826].

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