Skip to content

Commit 69263d0

Browse files
committed
Merge branch 't/12747'
2 parents c5ed166 + dff68f1 commit 69263d0

File tree

4 files changed

+36
-2
lines changed

4 files changed

+36
-2
lines changed

CHANGES.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,10 @@ CKEditor 4 Changelog
33

44
## CKEditor 4.4.7
55

6+
Fixed Issues:
7+
8+
* [#12747](http://dev.ckeditor.com/ticket/12747): [IE8-10] Fixed: Opening a drop-down for a specific selection when editor is maximized results in incorrect drop-down panel position.
9+
610
## CKEditor 4.4.6
711

812
**Security Updates:**

core/dom/element.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1425,9 +1425,9 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab
14251425
}
14261426

14271427
if ( needAdjustScrollAndBorders ) {
1428-
x = box.left + ( !quirks && $docElem.scrollLeft || body.$.scrollLeft );
1428+
x = box.left + ( quirks ? body.$.scrollLeft : $docElem.scrollLeft );
14291429
x -= clientLeft;
1430-
y = box.top + ( !quirks && $docElem.scrollTop || body.$.scrollTop );
1430+
y = box.top + ( quirks ? body.$.scrollTop : $docElem.scrollTop );
14311431
y -= clientTop;
14321432
}
14331433
} else {

tests/tickets/12747/1.html

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
<p>Some text</p>
2+
<p>Some text</p>
3+
<p>Some text</p>
4+
<textarea id="editor1">
5+
<p>Foo</p>
6+
<p>Bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar bar</p>
7+
</textarea>
8+
9+
<script>
10+
CKEDITOR.replace( 'editor1' );
11+
</script>

tests/tickets/12747/1.md

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
@bender-tags: 4.4.7, tc
2+
@bender-ui: collapsed
3+
@bender-ckeditor-plugins: wysiwygarea, toolbar, maximize, floatingspace, font
4+
5+
Note:
6+
7+
* Related bug was reproducible on IE8-10 only.
8+
* Make the selection precisely.
9+
10+
Steps:
11+
12+
1. Click the Maximize button.
13+
1. Using your mouse select text:
14+
* Start **right above** "Foo".
15+
* Finish below the second line.
16+
* This should result in selecting the entire content.
17+
1. Click the Font dropdown.
18+
19+
**Expected result**: Dropdown panel is visible and correctly positioned.

0 commit comments

Comments
 (0)