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.
100%
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
| 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 |
Updated by Wittkiel Gruppe over 6 years ago
- File ckeditor-maximized-chrome.png ckeditor-maximized-chrome.png added
- File ckeditor-maximized-firefox.png ckeditor-maximized-firefox.png added
- File ckeditor-panel.png ckeditor-panel.png added
- Subject changed from RTE CKeditor to RTE CKeditor top-positioning for maximize and combopanels is broken in browsers with webkit
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.
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.
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
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.
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;
}
Updated by Arne Bracht over 6 years ago
Thanks, worked for me. So it seems not to be from the ckeditor project.
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
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
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
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
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
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
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
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
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]*/
}
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
Updated by Benjamin Franzke about 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 480161c8d35475b1aec1f115c67a16ae301ab043.
Updated by
Updated by Benjamin Franzke about 2 years ago
- Related to Bug #97144: Slow module scrolling in Google Chrome on Linux added