Bug #80116

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

Added by Benjamin no-lastname-given over 4 years ago. Updated almost 3 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

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
Has duplicate TYPO3 Core - Bug #83332: CKEditor: Browser jumps to top when selecting a style in TYPO3 8.7.9Closed2017-12-14

Actions
#1

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

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

Updated by Benjamin no-lastname-given over 4 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.

#3

Updated by Riccardo De Contardi over 4 years ago

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

#4

Updated by Frank Naegler over 4 years ago

confirmed, chrome is ok, safari the issue still exists

#5

Updated by Frank Naegler over 4 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.

#6

Updated by Riccardo De Contardi almost 4 years ago

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

#7

Updated by Benjamin Franzke almost 4 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/';

#8

Updated by Riccardo De Contardi almost 4 years ago

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

Updated by Christian Hackl almost 4 years ago

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

MAC 10.12.6 and Chrome 61.0.3163.100

#10

Updated by Wittkiel Gruppe almost 4 years ago

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

#11

Updated by Riccardo De Contardi almost 4 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?

#12

Updated by Georg Ringer almost 4 years ago

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

Updated by Francois Suter almost 4 years ago

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

#14

Updated by Arne Bracht Bracht almost 4 years ago

Works for me, too.

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

#15

Updated by Arne Bracht Bracht almost 4 years ago

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

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

#17

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

#18

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

#19

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

#20

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

#21

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

#22

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

#23

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

#24

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

#25

Updated by Georg Ringer almost 4 years ago

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

Updated by Ernesto Baschny over 3 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.

#27

Updated by Riccardo De Contardi over 3 years ago

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

Updated by Susanne Moog over 3 years ago

  • Target version changed from 9.0 to 9.2
#29

Updated by Gerrit Code Review over 3 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

#30

Updated by Gerrit Code Review over 3 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

#31

Updated by Gerrit Code Review over 3 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

#32

Updated by Gerrit Code Review over 3 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

#33

Updated by Gerrit Code Review over 3 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

#34

Updated by Gerrit Code Review over 3 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

#35

Updated by Gerrit Code Review over 3 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

#36

Updated by Gerrit Code Review over 3 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

#37

Updated by Gerrit Code Review over 3 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

#38

Updated by Gerrit Code Review over 3 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

#39

Updated by Gerrit Code Review over 3 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

#40

Updated by Gerrit Code Review over 3 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

#41

Updated by Gerrit Code Review over 3 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

#42

Updated by Gerrit Code Review over 3 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

#43

Updated by Benjamin Franzke over 3 years ago

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

Updated by Benni Mack almost 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF