search

readthedocs-fr / bin-server

Un outil pour héberger des snippets de code et les partager via une URL.
https://bin.readthedocs.fr
MIT License
14 stars 15 forks source link

Bouton `signalé` passe au dessus de la barre de défilement. #136

Open enimaloc opened 3 years ago

enimaloc commented 3 years ago

Lorsque un bin devient trop grand la barre de défilement apparaît sur la page, ce cas-ci n'est pas gérer pour le bouton signalé qui passe au-dessus de la barre de défilement.

Nauhai commented 3 years ago

À compter que c'est aussi le cas pour les boutons de controls image

Mesteery commented 3 years ago

Pour le boutons de contrôles, la scrollbar est présente inutilement des fois, c'est fix par #116

Nauhai commented 3 years ago

Le problème vient du fait que les boutons de contrôles ainsi que le bouton de signalement sont dans un conteneur différent de celui qui a les scroll-bar :

Une solution serait de déplacer les boutons dans les conteneurs dont les scrollbars appartiennent, mais ça sera compliqué avec .highlight étant donné qu'il est généré par Pygments.

Une autre solution serait d'augmenter les padding/margin des boutons jusqu'à avoir une distance convenable de la scrollbar. Cependant les padding/margin resteront tels même si les scrollbars ne sont pas présentes, et je pense que la taille des scrollbars dépend du navigateur/appareil.

Est-ce que vous avez d'autres solutions ?

Mesteery commented 3 years ago

Le problème vient du fait que les boutons de contrôles ainsi que le bouton de signalement sont dans un conteneur différent de celui qui a les scroll-bar :

Non, c'est juste qu'ils sont en position fixed / absolute.

Nauhai commented 3 years ago

Non, c'est juste qu'ils sont en position fixed / absolute.

Oui, et bah s'ils étaient dans le même conteneur que celui qui a les scrollbars (avec position: relative sur le conteneur parent) on n'aurait pas ce problème là : image

Nauhai commented 3 years ago

Pour le boutons de contrôles, la scrollbar est présente inutilement des fois, c'est fix par #116

Quand je fetch ta PR c'est pas fix, il suffit juste d'enlever le overflow-x: scroll (d'ailleurs il fout quoi ici ???).

Mesteery commented 3 years ago

oui en fait je m'étais trompé cette PR ne le fix pas. Sinon, si, ils sert. (pour fix faut juste mettre overflow-x: auto au lieu de scroll)