dklemmt / contao_dk_mmenu

Implementation of mmenu as extension for Contao Open Source CMS
15 stars 16 forks source link

Springen der Seite beim Einblenden des Menüs #37

Closed saress closed 5 years ago

saress commented 5 years ago

Hallo Fritz,

mir ist noch aufgefallen, dass das Einblenden der Navigation dazu führt, dass die Scrollbar ausgeblendet wird und die Seiten daher ein wenig "springen", wenn Sie bereits eine Scrollbar haben.

Konkret hängt das vermutlich mit folgender Anweisung zusammen:

.mm-wrapper_blocking body { overflow: hidden; }

die beim Öffnen des Menüs greift. Ändert man das auf:

overflow:scroll;

verschwindet das Springen. Ich hab jetzt aber nicht getestet, ob das irgendwelche anderen Side-Effekte bei Seiten hat, die generell keine Scrollbar haben / brauchen.

Beste Grüße Sares

fritzmg commented 5 years ago

Hm, solche Issues müsstest du hier posten: https://github.com/FrDH/mmenu-js

saress commented 5 years ago

OK, danke, ich werde vielleicht zuerst noch schauen, ob das jemand anderem auch schon aufgefallen ist :-)

fritzmg commented 5 years ago

Ich kann das auch nachvollziehen. Ist ein typisches Windows Problem. Das Contao back end hat im Prinzip ein verwandtes Problem ;).

Aber die non-Windows User lassen sich dadurch auch schwer überzeugen, dass man das evt. anders machen muss.

saress commented 5 years ago

;-)

Ich habe folgendes gefunden:

https://github.com/FrDH/mmenu-js/issues/752

Die Option habe ich jetzt in Deinem Modul auf die Schnelle nicht gefunden und das CSS müsste man dann wohl auch etwas "modifizieren", insgesamt scheint das Problem aber bereits bekannt zu sein.

fritzmg commented 5 years ago

Einfach

html.mm-opened #page {
    overflow: auto !important;
}

In dein eigenes CSS hinzufügen.

saress commented 5 years ago

Danke, ja das hab ich gesehen, werde ich morgen dann gleich mal ausprobieren, irgendwie stört mich das nämlich ein wenig :-)

saress commented 5 years ago

Letztendlich hat folgendes funktioniert:

.mm-wrapper-blocking body { overflow: scroll; }

Wie ich gerade gesehen habe, gibt es auch bereits wieder einen neuen Release 8,2.2. Ist Deine Version "angepasst", oder kann man einfach die neue Version einspielen?

Beste Grüße Sares

fritzmg commented 5 years ago

Könntest du prinzipiell einspielen - geht halt beim nächsten Update wieder verloren. Aber das nächste Update wird vielleicht eh wieder die neueste Version enthalten ;)

saress commented 5 years ago

Danke Fritz, dann werde ich eher auf Dein Update warten, soweit scheint ja alles was ich brauche zu funktionieren und es ist daher nicht dringend.

fritzmg commented 5 years ago

Version 3.0.2 benutzt nun Version 8.2.3 von mmenu. @saress ist dieses Ticket noch relevant?

saress commented 5 years ago

Hallo Fritz,

da ich denke, dass das eine individuelle Sache ist, die man einfach selbst lösen kann und die vielleicht nicht jeden stört, kann man das Ticket sicher schließen.

Danke Fritz und beste Grüße Sares