scss: include reponsive menu mask display option (#72654) #87
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/72654-show-menu-mask-option"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Reprise de ce qu'a fait ThomasJ pour le CD06.
@ -339,2 +341,4 @@
@if $responsive-menu == left-to-right {
background: transparent;
@if $responsive-menu-use-mask == true {
@media ($max-mobile-viewport) {
Attention, la nav a son propre breakpoint : $nav-mobile-limit.
Et tu es déjà dans une condition
@media screen and (max-width: $nav-mobile-limit) {
initié ligne 19647610140eb
to11fcf04c2c
@ -340,1 +342,4 @@
background: transparent;
@if $responsive-menu-use-mask == true {
@media (max-width: $nav-mobile-limit) {
~ div.gru-nav-mask {
Je déplacerais ce code après
&+ ul
pour rester cohérent avec l'organisation présente@ -35,6 +35,8 @@ $nav-mobile-menu-item-hover-background: $nav-item-hover-background !default;
$nav-mobile-menu-item-hover-color: $nav-item-hover-color !default;
$nav-item-spacing: 0px !default;
$responsive-menu: top-to-bottom !default; // or left-to-right
$responsive-menu-use-mask: false !default;
Pourquoi pas simplement
$responsive-menu-mask
?C'est ma pérception de la chose: comme c'est un booléen il faut un verbe.
Mais ça me va de faire sans.
@ -341,0 +343,4 @@
@if $responsive-menu-use-mask == true {
@media (max-width: $nav-mobile-limit) {
~ div.gru-nav-mask {
display: block;
Pourquoi
display:block
, je ne vois nul part undisplay:none
avant.11fcf04c2c
to7d7a211211
Remarques prises en compte.
@ -362,6 +364,19 @@ div.menucell {
} @else if $responsive-menu == left-to-right {
transform: translateX(0);
}
@if $responsive-menu-mask == true {
Je proposais de positionner en dehors de ul, juste après
Et il manque la condition left-to-right
@if ($responsive-menu-mask == true and $responsive-menu == left-to-right)
@ -363,2 +365,4 @@
transform: translateX(0);
}
@if $responsive-menu-mask == true {
@media (max-width: $nav-mobile-limit) {
À supprimer, (c'est inutile tu es déjà dans une condition nav mobile)
7d7a211211
to22dd788aae
Remarques prises en compte.
Pourquoi c'est une option et pas juste l'unique comportement ?
Par méfiance je préfère que ce soit une option désactivée par défaut.