Task #84522

Fix for Backend not scrollable

Added by Forger Service 9 months ago. Updated 3 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2018-03-23
Due date:
% Done:

0%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Some Extensions are not scrollabe any more since the iframe #typo3-contentIframe has added a scrolling="no"
This issue was automatically created from https://github.com/TYPO3/TYPO3.CMS/pull/103


Related issues

Related to TYPO3 Core - Bug #84476: Backend Scrolling own Extension Needs Feedback 2018-03-19
Related to TYPO3 Core - Bug #84264: Backend not scrollable Closed 2018-03-14
Related to TYPO3 Core - Bug #84263: Backend Content not scrollable if only DebuggerUtility::var_dump New 2018-03-14

History

#1 Updated by Gerrit Code Review 9 months 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/56439

#2 Updated by Riccardo De Contardi 9 months ago

I also had this problem (not scrollable view) when I got an Uncaught Exception screen on 9.2 - the exception output is very long and not scrollable. The proposed fix seemed to work, at least on Chrome. I did not test it on other browsers or handheld devices.

#3 Updated by Benjamin Kott 9 months ago

i´ve abandoned the patch because the change is too risky and part of a recent correction we did that fixes the scrolling on ios devices
https://github.com/TYPO3/TYPO3.CMS/commit/104cee47f3578e7b2a45d7cc8059fa5c75ba63af#diff-c8aa135a7bae4ca077fdf5a4c1fe1edd

we need more information on how to reproduce this issue on a clean master and 8.7.
currently not aware of any of these symptoms with an unmodified backend.

#4 Updated by Clemens Riccabona 9 months ago

Can confirm this no-scroll problem with:
  • PHP 7.2.x
  • TYPO3 8.7.12
  • Firefox 59.0.1_x64 on openSuSE 43.3

I am able to reproduce this e.g. with extension phpmyadmin.

Proposed patch works as expected on the defined environment.

Side note concerning "risky iphone changes": I would love, if TYPO3 backend would stick with "desktop first" development approach, as I guess the system is (and will always(tm) be) mainly used with Desktops.

EDIT: can confirm bug and patch also on TYPO3 8.7.11, the rest of the setup is the same.

#5 Updated by Ralf Merz 9 months ago

Hi guys,

I can confirm broken scrolling on v8.7.11 and v8.7.12. Just tried a rollback to 8.7.10 which makes scrolling possible again.
The patch seems to work for me, too.
My Browser is newest Safari on Mac.

Regards
merzilla

#6 Updated by Stefan Froemken 8 months ago

I don't think that this is a TYPO3 bug. TYPO3 BE is based either on DocumentTemplate or ModuleTemplate.

ModuleTemplate calls:
$this->pageRenderer->setXmlPrologAndDocType('<!DOCTYPE html>');

DocumentTemplate calls:
$headerStart = '<!DOCTYPE html>';
$this->pageRenderer->setXmlPrologAndDocType($headerStart);

So DOCTYPE is HTML5 in both cases.

The scrolling Attribute is outdated in HTML5. So it will not be interpreted.

As PageRenderer is a SingletonInterface it is possible for Extension Developers to override this setting from nearly everywhere.
Please check the HTML sourcecode of main Doc. It must be <!DOCTYPE html>
If your DocType is different, something (f.e. an extension) has overriden this setting.

Stefan

#7 Updated by Wolfgang Wagner 8 months ago

We had this problem in a TYPO3 8.7.12 of a hosting customer, who used the extension ws_t3bootstrap (a commercial Extension, afaik).

The problem was caused by a CSS definition in the file
EXT:ws_t3bootstrap/Resources/Public/CSS/Backend/mods.css

body {height:auto;}

This definition has also an impact on the body tag within the iframe. Without height:auto; everything works fine.

#8 Updated by Gert-jan Dikkescheij 8 months ago

I used https://extensions.typo3.org/extension/ckeditor_fix/ in many of my intalations, this extension causes this same issue (from v8.7.11 on), and has, as wolfgang reported, body {height:auto;} in the following file:

EXT:ckeditor_fix/Resources/Public/Backend/Css/Skin/t3skin_override.css

furthermore did I find that the issues that this extension fixed are resolved anyhow ...

#9 Updated by Aimeos no-lastname-given 8 months ago

We have the same issue with the Aimeos backend in TYPO3 but we found none of the problematic CSS styles in our code. We use Twitter Bootstrap (4.1) in the backend.

It only works if we explicitely set:

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

#10 Updated by Riccardo De Contardi 7 months ago

  • Related to Bug #84476: Backend Scrolling own Extension added

#11 Updated by Riccardo De Contardi 7 months ago

  • Related to Bug #84264: Backend not scrollable added

#12 Updated by Riccardo De Contardi 7 months ago

  • Related to Bug #84263: Backend Content not scrollable if only DebuggerUtility::var_dump added

#13 Updated by Riccardo De Contardi 7 months ago

  • Category set to Backend User Interface

#14 Updated by Clemens Riccabona 6 months ago

Bug still present in TYPO3 v 8.7.15

#15 Updated by Clemens Riccabona 6 months ago

Still present in 8.7.16

#16 Updated by Marc Maeder 4 months ago

Still present in 8.7.17 and 8.7.18

#17 Updated by Micha 3 months ago

Changing Line 17 in
in File

 sysext\backend\Resources\Private\Templates\Backend\Main.html

<iframe name="list_frame" id="typo3-contentIframe" scrolling="no" ...

changed to
<iframe name="list_frame" id="typo3-contentIframe" scrolling="auto" ...

and removing the cached Version from "typo3temp/" did work for me.

Also available in: Atom PDF