Project

General

Profile

Actions

Bug #80116

closed

rte_ckeditor displaces dropdown overlays and jumps to top of page on crome/safari

Added by Benjamin no-lastname-given about 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
Start date:
2017-03-03
Due date:
% Done:

100%

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

Description


Related issues 6 (0 open6 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 #82928: RTE (ckeditor) menus broken in Chrome 61+62Closed2017-11-06

Actions
Related to TYPO3 Core - Bug #82780: RTE CKeditor top-positioning for maximize and combopanels is broken in browsers with webkitClosed2017-10-17

Actions
Related to TYPO3 Core - Bug #83379: Dropdown (paragraph f.e.) in RTE(htmlarea) instantly closes after openingClosed2017-12-18

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 #1

Updated by Benjamin no-lastname-given about 7 years ago

  • Description updated (diff)
  • Category changed from Backend JavaScript to RTE (rtehtmlarea + ckeditor)
Actions #2

Updated by Benjamin no-lastname-given about 7 years ago

  • Assignee set to Benni Mack

Hi Benni,
i have seen you are handling the other CKEditor issues, so i also assign this one to you.

Actions #3

Updated by Riccardo De Contardi almost 7 years ago

That still affects Safari but not Chrome anymore. Can someone confirm?

Actions #4

Updated by Frank Nägler almost 7 years ago

confirmed, chrome is ok, safari the issue still exists

Actions #5

Updated by Frank Nägler almost 7 years ago

  • Target version changed from 8 LTS to 9.0

looks like this is a known bug. try this google search:
https://www.google.de/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=ckeditor+dropdown+jumpx%x%*

I think this should be fixed but is not critical for the upcoming release.

Actions #6

Updated by Riccardo De Contardi over 6 years ago

I found this issue again with Chrome (on Mac), both affects 8.7.7 and latest master 9.0.0-dev

Actions #7

Updated by Benjamin Franzke over 6 years ago

This is due TYPO3's css setting body height to 100%:
https://github.com/TYPO3/TYPO3.CMS/blob/8.7.8/Build/Resources/Public/Sass/scaffold/_scaffold.scss#L36

This code was introduced in a giant commit https://github.com/TYPO3/TYPO3.CMS/commit/9b7ed3aec253bad5221407834903f4231bb6e2b4 without further clarification what "Browser Rebounce" is supposed to be. It seems that the buggy code is copied from https://stackoverflow.com/a/28985569, as they are talking about a "bounce" effect, which Kay has probably rephrased to "Rebounce".

Don't know, whether it would be accepted to just remove those lines (body height 100% is idiotic), but I think desktop Chrome is far more important than mobile safari, therefore i've fixed it with a t3skin override [1]:

/* TYPO3 uses html {overflow: hidden, height: 100%} body {height: 100%;overflow:auto}
 * to prevent some "Browser Rebounce" – with no further
 * explanation what that is.
 * That "fix" causes a bugs in chrome because:
 *  * ckeditor context/popup menu cause the page to scroll up as body is only
 *    100% in height and those elements are placed absolutly in the <body> tag
 *  * The scrollbar is overlayed by the topbar
 */

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

[1] Put above css into an extension in a file called Resources/Public/Backend/Css/Skin/t3skin_override.css and add to ext_tables.php:

$TBE_STYLES['skins'][$_EXTKEY]['name'] = $_EXTKEY;
$TBE_STYLES['skins'][$_EXTKEY]['stylesheetDirectories']['structure'] = 'EXT:' . $_EXTKEY . '/Resources/Public/Backend/Css/Skin/';

Actions #8

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 #9

Updated by Christian Hackl over 6 years ago

@Benjamin Fischer Franzke
sry your fix don't work here:

MAC 10.12.6 and Chrome 61.0.3163.100

Actions #10

Updated by Wittkiel Gruppe over 6 years ago

The override-fix works for us with Win10 and Chrome 62.

Actions #11

Updated by Riccardo De Contardi over 6 years ago

@Christian Hackl (mine is just a wild guess, I have not tested yet and don't know if you have already done that) could you test it again, but removing the browser cache?

Actions #12

Updated by Georg Ringer over 6 years ago

  • Related to Bug #82928: RTE (ckeditor) menus broken in Chrome 61+62 added
Actions #13

Updated by Francois Suter over 6 years ago

The fix works for me on Mac and Chrome 62 (with TYPO3 8.7.8).

Actions #14

Updated by Arne Bracht over 6 years ago

Works for me, too.

Would be very nice to fix this in the next TYPO3 V8 release.

Actions #15

Updated by Arne Bracht over 6 years ago

  • Related to Bug #82780: RTE CKeditor top-positioning for maximize and combopanels is broken in browsers with webkit added
Actions #16

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New 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 #17

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 #18

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 #19

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 #20

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 #21

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 #22

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 #23

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 #24

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 #25

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 #26

Updated by Ernesto Baschny about 6 years ago

The bugfix as an extension while we have no core fix available: https://extensions.typo3.org/extension/ckeditor_fix/ - might "break the iPad" experience if that matters to you.

Actions #27

Updated by Riccardo De Contardi about 6 years ago

  • Related to Bug #83379: Dropdown (paragraph f.e.) in RTE(htmlarea) instantly closes after opening added
Actions #28

Updated by Susanne Moog about 6 years ago

  • Target version changed from 9.0 to 9.2
Actions #29

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 #30

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 #31

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 #32

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 #33

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 #34

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 #35

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 #36

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 #37

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 #38

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 #39

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 #40

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 #41

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 #42

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 #43

Updated by Benjamin Franzke about 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #44

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions #45

Updated by Benjamin Franzke about 2 years ago

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

Also available in: Atom PDF