Project

General

Profile

Actions
Copy link

Bug #80116

closed

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

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

Status:
Closed
Priority:
Must have
Assignee:
Benni Mack
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
9.2
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 open — 6 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
Copy link
 #1

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

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

Updated by Benjamin no-lastname-given over 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
Copy link
 #3

Updated by Riccardo De Contardi over 7 years ago

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

Actions
Copy link
 #4

Updated by Frank Nägler over 7 years ago

confirmed, chrome is ok, safari the issue still exists

Actions
Copy link
 #5

Updated by Frank Nägler over 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
Copy link
 #6

Updated by Riccardo De Contardi about 7 years ago

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

Actions
Copy link
 #7

Updated by Benjamin Franzke about 7 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
Copy link
 #8

Updated by Riccardo De Contardi about 7 years ago

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

Updated by Christian Hackl about 7 years ago

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

MAC 10.12.6 and Chrome 61.0.3163.100

Actions
Copy link
 #10

Updated by Wittkiel Gruppe about 7 years ago

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

Actions
Copy link
 #11

Updated by Riccardo De Contardi about 7 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
Copy link
 #12

Updated by Georg Ringer about 7 years ago

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

Updated by Francois Suter about 7 years ago

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

Actions
Copy link
 #14

Updated by Arne Bracht about 7 years ago

Works for me, too.

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

Actions
Copy link
 #15

Updated by Arne Bracht about 7 years ago

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

Updated by Gerrit Code Review about 7 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
Copy link
 #17

Updated by Gerrit Code Review about 7 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
 #18

Updated by Gerrit Code Review about 7 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
 #19

Updated by Gerrit Code Review about 7 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
 #20

Updated by Gerrit Code Review about 7 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
 #21

Updated by Gerrit Code Review about 7 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
 #22

Updated by Gerrit Code Review about 7 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
 #23

Updated by Gerrit Code Review almost 7 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
 #24

Updated by Gerrit Code Review almost 7 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
 #25

Updated by Georg Ringer almost 7 years ago

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

Updated by Ernesto Baschny almost 7 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
Copy link
 #27

Updated by Riccardo De Contardi almost 7 years ago

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

Updated by Susanne Moog almost 7 years ago

  • Target version changed from 9.0 to 9.2
Actions
Copy link
 #29

Updated by Gerrit Code Review almost 7 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
 #30

Updated by Gerrit Code Review almost 7 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
 #31

Updated by Gerrit Code Review almost 7 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
 #32

Updated by Gerrit Code Review almost 7 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
 #33

Updated by Gerrit Code Review almost 7 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
 #34

Updated by Gerrit Code Review almost 7 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
 #35

Updated by Gerrit Code Review almost 7 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
 #36

Updated by Gerrit Code Review almost 7 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
 #37

Updated by Gerrit Code Review almost 7 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
 #38

Updated by Gerrit Code Review almost 7 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
 #39

Updated by Gerrit Code Review almost 7 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
 #40

Updated by Gerrit Code Review almost 7 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
 #41

Updated by Gerrit Code Review almost 7 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
 #42

Updated by Gerrit Code Review almost 7 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
 #43

Updated by Benjamin Franzke almost 7 years ago

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

Updated by Benni Mack about 6 years ago

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

Updated by Benjamin Franzke over 2 years ago

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

Also available in: Atom PDF