Projet

Général

Profil

Bug #56557

les barres du menu mobile burger sont "vides"

Ajouté par Serghei Mihai il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
01 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

En mobile, sur un écran à haute résolution ou en zoomant sur la page on voit que les barres du menu burger sont "vides".


Fichiers

burger-cd06.png (124 ko) burger-cd06.png Serghei Mihai, 01 septembre 2021 11:43
Fontenay-sous-Bois.burger.png (157 ko) Fontenay-sous-Bois.burger.png Serghei Mihai, 01 septembre 2021 11:43
0001-scss-fill-burgers-bars-background-56557.patch (673 octets) 0001-scss-fill-burgers-bars-background-56557.patch Serghei Mihai, 02 septembre 2021 11:14
0001-scss-refactor-mobile-burger-menu-bars-appearance-565.patch (3,63 ko) 0001-scss-refactor-mobile-burger-menu-bars-appearance-565.patch Serghei Mihai, 02 septembre 2021 12:35
0001-scss-refactor-mobile-burger-menu-bars-appearance-565.patch (3,61 ko) 0001-scss-refactor-mobile-burger-menu-bars-appearance-565.patch Serghei Mihai, 14 septembre 2021 10:51
arles.png (83,6 ko) arles.png Serghei Mihai, 16 septembre 2021 15:22
arles-burger-btn.png (48,4 ko) arles-burger-btn.png Thomas Jund (congés, retour le 29/04), 16 septembre 2021 17:14

Historique

#2

Mis à jour par Serghei Mihai il y a plus de 2 ans

Proposition simple de définir une couleur de fond identique à celle des bordures.

#3

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
  • Assigné à mis à Benjamin Dauvergne

Je ne vois pas d'autre solution.

#4

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans

  • Statut changé de Solution validée à Solution proposée
  • Assigné à Benjamin Dauvergne supprimé

Je serais plus partisan de mettre

border-top: 2px solid $nav-button-color;

Comme ça on a une vraie maitrise sur la hauteur de la barre.
(Mais ça peux impacter des thèmes)

#5

Mis à jour par Serghei Mihai il y a plus de 2 ans

  • Statut changé de Solution proposée à En cours

Thomas Jund a écrit :

Je serais plus partisan de mettre

[...]

Cela fera des barres plus épaisses. On passe de 2px à 4.

#6

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans

non, parce qu'on fixe sur border-top à la place de border.

#7

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Thomas Jund a écrit :

non, parce qu'on fixe sur border-top à la place de border.

Ça me va aussi.

#8

Mis à jour par Serghei Mihai il y a plus de 2 ans

Thomas Jund a écrit :

non, parce qu'on fixe sur border-top à la place de border.

Ça fonctionne. Mais cela impactera des thèmes. On laisse pour la release suivante.

#9

Mis à jour par Serghei Mihai il y a plus de 2 ans

Patch à jour avec les modifications dans les thèmes impactés.

#11

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans

Sur les thèmes lille & co et Arles-2020, un border-radius est appliqué sur les barres du burger btn (j'ai croisé ça récement en bossant sur #56855).

publik-base-theme/static/arles-2020/style.css
  div.gru-nav .gru-nav-button .icon-bar
    border-radius 10px 0 0 10px
    { line: 4348, column: 5 }

Du coup, basculer en `border-top` la gestion de la hauteur des barres supprime cette optin graphique.

#12

Mis à jour par Serghei Mihai il y a plus de 2 ans

Pourtant le radius est bien pris en compte (exemple sur Arles)

#13

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans

Moi il me reste un demi border (ce qui me parait logique).

#14

Mis à jour par Serghei Mihai il y a plus de 2 ans

Ça me paraît logique aussi. J'ai accusé mon Firefox mais Chromium me donne le même rendu.
Bref, pour l'instant je n'ai pas d'autres idées que revenir sur le patch initial: attribuer la même couleur de fond que celle de la bordure.

#15

Mis à jour par Serghei Mihai il y a plus de 2 ans

  • Statut changé de Solution proposée à Fermé

Traité dans #59319.

Formats disponibles : Atom PDF