Project

General

Profile

Actions
Copy link

Bug #82780

closed

RTE CKeditor top-positioning for maximize and combopanels is broken in browsers with webkit

Added by Arne Bracht over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Won't have this time
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2017-10-17
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.1
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Fullscreen mode works not correct after scrolling down an then press the button to maximize. It calculate the css class "cke_maximized" to negativ values for "top".

Affected is Chrome version Version 61.0.3163.100 (Offizieller Build) (32-Bit), tried also 64 bit with the same result.
Firefox is everything ok.

The problem is in \typo3\sysext\rte_ckeditor\Resources\Public\JavaScript\Contrib\ckeditor.js

It calculate the css class "cke_maximized" to negativ values for "top".

Test sets cke_maximized {top: 0px;}, but right click menus are now on false positions:

- y.addClass("cke_maximized");f();w=y.getDocumentPosition();y.setStyles({left:-1*w.x+"px",top:-1*w.y+"px"});
+ y.addClass("cke_maximized");f();w=y.getDocumentPosition();y.setStyles({left:-1*w.x+"px",top:"0px"});

Shows what i mean:
https://typo3.slack.com/files/U5STQ9VN2/F7HSA10U8/f__rderschule_landau__typo3_cms_8.7.8__-_google_chrome_15.10.2017_15_13_41.mp4

Files

Download all files
ckeditor-maximized-chrome.png (45.1 KB) ckeditor-maximized-chrome.png Wittkiel Gruppe, 2017-10-18 12:01
ckeditor-maximized-firefox.png (58.4 KB) ckeditor-maximized-firefox.png Wittkiel Gruppe, 2017-10-18 12:01
ckeditor-panel.png (58 KB) ckeditor-panel.png Wittkiel Gruppe, 2017-10-18 12:01

Related issues 4 (0 open — 4 closed)

Related to TYPO3 Core - Bug #82841: page scroll up when click on drop down button of ckeditorClosed2017-10-23

Actions
Related to TYPO3 Core - Bug #80116: rte_ckeditor displaces dropdown overlays and jumps to top of page on crome/safariClosedBenni Mack2017-03-03

Actions
Related to TYPO3 Core - Bug #97144: Slow module scrolling in Google Chrome on LinuxClosedBenjamin Franzke2022-03-08

Actions
Has duplicate TYPO3 Core - Bug #83332: CKEditor: Browser jumps to top when selecting a style in TYPO3 8.7.9Closed2017-12-14

Actions
Actions
Copy link Download all files
 #1

Updated by Wittkiel Gruppe over 6 years ago

Same problems for us with:
TYPO3 8.7.8
Chrome 62.0.3202.62 AND Opera 48.0.2685.39
Windows 10

After pressing the button "Maximize" top position is incorrect in Chrome and Opera (webkit):

In Firefox the top position is correct:

Also in Chrome and Opera the positioning of combopanel is wrong:

Seems to be an error in function "getDocumentPosition" in typo3/sysext/rte_ckeditor/Resources/Public/JavaScript/Contrib/ckeditor.js.

Actions
Copy link
 #2

Updated by Wouter Wolters over 6 years ago

  • Status changed from New to On Hold
  • Priority changed from Should have to Won't have this time
  • Target version deleted (next-patchlevel)

TYPO3 can not change this file, as CKeditor is a 3rd party extension TYPO3 uses.

Actions
Copy link
 #3

Updated by Riccardo De Contardi over 6 years ago

  • Related to Bug #82841: page scroll up when click on drop down button of ckeditor added
Actions
Copy link
 #4

Updated by Susanne Moog over 6 years ago

  • Status changed from On Hold to Rejected

Please report your issue at the ckeditor project at https://dev.ckeditor.com/wiki/Bugs as the error is in their code.

Actions
Copy link
 #5

Updated by Wittkiel Gruppe over 6 years ago

The solution in #80116 (comment Nr. 7) works for us in Chrome 62. Seems to be a problem with these lines:

//
// Prevent Browser Rebounce
//
html {
    height: 100%;
    overflow: hidden;
}

body {
    height: 100%;
    overflow: auto;
}

https://github.com/TYPO3/TYPO3.CMS/blob/TYPO3_8-7/Build/Resources/Public/Sass/scaffold/_scaffold.scss#L35-L46

This temporary override (TBE_STYLES) fixed the problem in Chrome:


html {
    overflow: auto;
}

body {
    height: auto;
    min-height: 100%;
    overflow: visible;
}

Actions
Copy link
 #6

Updated by Arne Bracht over 6 years ago

Thanks, worked for me. So it seems not to be from the ckeditor project.

Actions
Copy link
 #7

Updated by Arne Bracht over 6 years ago

  • Related to Bug #80116: rte_ckeditor displaces dropdown overlays and jumps to top of page on crome/safari added
Actions
Copy link
 #8

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Rejected to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #9

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #10

Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #11

Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #12

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54648

Actions
Copy link
 #13

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54648

Actions
Copy link
 #14

Updated by Gerrit Code Review over 6 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #15

Updated by Gone With the Wind over 6 years ago

The iPad/iPhones seem to be problematic (googled for "ipad overflow auto problem"...).
I do not have an ipad, but can someone try this, please:

/*
 * Make CKEditor work on iPad, too.
 * overflow:auto seems to work differently on Safari and Mobile Safari.
 * 
 * [1] Add a special webkit rule for touch interaction
 * [2] Use "scroll" instead of "auto".
 */ 
html {
    -webkit-overflow-scrolling:touch; /*[1]*/
    overflow: scroll;  /*[2]*/
}
Actions
Copy link
 #16

Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54648

Actions
Copy link
 #17

Updated by Gerrit Code Review over 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #18

Updated by Georg Ringer over 6 years ago

  • Has duplicate Bug #83332: CKEditor: Browser jumps to top when selecting a style in TYPO3 8.7.9 added
Actions
Copy link
 #19

Updated by Gerrit Code Review about 6 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #20

Updated by Gerrit Code Review about 6 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #21

Updated by Gerrit Code Review about 6 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #22

Updated by Gerrit Code Review about 6 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #23

Updated by Gerrit Code Review about 6 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54620

Actions
Copy link
 #24

Updated by Gerrit Code Review about 6 years ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #25

Updated by Gerrit Code Review about 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #26

Updated by Gerrit Code Review about 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #27

Updated by Gerrit Code Review about 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #28

Updated by Gerrit Code Review about 6 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #29

Updated by Gerrit Code Review about 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #30

Updated by Gerrit Code Review about 6 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #31

Updated by Gerrit Code Review about 6 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55647

Actions
Copy link
 #32

Updated by Gerrit Code Review about 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55657

Actions
Copy link
 #33

Updated by Benjamin Franzke about 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #34

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link
 #35

Updated by Benjamin Franzke about 2 years ago

  • Related to Bug #97144: Slow module scrolling in Google Chrome on Linux added
Actions
Copy link

Also available in: Atom PDF