Development #14830
rajouter les styles pour le menu "sandwich"
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans
- Lié à Development #14010: Possibilité d'abonner à une categorie à partir d'une liste d'adresses email fournie dans un CSV ajouté
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans
- Fichier 0001-styles-for-sandwich-menu-14830.patch 0001-styles-for-sandwich-menu-14830.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 7 ans
Il manque la part de js pour rendre ça opérationnel; ce qui m'amène aussi à penser qu'il ne faut pas mettre les mêmes classes que dans combo parce que sinon on fait se trouver avec deux .toggle() exécutés lors du clic.
Je propose de taper comme classes extra-actions-menu et extra-actions-menu-opener. Il faudrait aussi que le menu soit caché par défaut (c'est fait avec un attribut style dans combo, pas top).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans
- Fichier 0001-styles-js-for-sandwich-menu-14830.patch 0001-styles-js-for-sandwich-menu-14830.patch ajouté
Ok
Mis à jour par Frédéric Péters il y a plus de 7 ans
"styles js for "sandwich" menu" pas top du tout ce message.
J'imagine plutôt quelque chose façon « add support for a 'sandwich' menu for extra actions » (qui serait même idéalement étendu par une description longue reprenant le markup attendu).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans
- Fichier 0001-styles-js-for-sandwich-menu-14830.patch ajouté
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans
- Fichier
0001-styles-js-for-sandwich-menu-14830.patchsupprimé
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans
- Fichier 0001-add-support-for-extra-actions-in-sandwich-menu-14830.patch 0001-add-support-for-extra-actions-in-sandwich-menu-14830.patch ajouté
Mauvais patch, désolé.
Mis à jour par Frédéric Péters il y a environ 7 ans
À jouer un peu, j'en profiterais pour faire mieux que ce qu'il y a dans combo, ne pas détourner de caractère ☰, ne rien mettre à la place dans le markup, mais utiliser FontAwesome ainsi :
.menu-opener::before {
content: "\f142";
font-family: FontAwesome;
font-size: 140%;
padding: 0 0.1ex 0 1ex;
}
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
- Fichier 0001-add-support-for-extra-actions-in-sandwich-menu-14830.patch 0001-add-support-for-extra-actions-in-sandwich-menu-14830.patch ajouté
- Fichier Screenshot_2017-02-07_17-09-01.png Screenshot_2017-02-07_17-09-01.png ajouté
Ok.
Le rendu dans la capture jointe.
Mis à jour par Frédéric Péters il y a environ 7 ans
Faudra faire d'autres trucs, le menu ouvert avec un seul élément c'est vraiment pas terrible. (mais bon, peut-être plus tard)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
Voici l'exemple sur Combo, avec plusieurs elements.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
- Lié à Development #14837: redefinir les styles du menu "sandwich" dans le backoffice ajouté
Mis à jour par Frédéric Péters il y a environ 7 ans
Oui, comme je disais, c'est pas terrible, il faudrait arriver à visuellement lier le "bouton" au menu qui apparait, etc. (et je disais "mais bon, peut-être plus tard")
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
Ok, faisons ça plus tard.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
- Priorité changé de Normal à Haut
Priorité haute pour la recette de Meyzieu.
Mis à jour par Frédéric Péters il y a environ 7 ans
- Fichier gadjo-kebab.png gadjo-kebab.png ajouté
- Fichier 0001-add-support-for-extra-actions-in-sandwich-menu-14830.patch 0001-add-support-for-extra-actions-in-sandwich-menu-14830.patch ajouté
Le reste du fichier est indenté avec des tabulations.
content: "\f142";, quand on fait une référence ainsi, il faut inclure le nom de l'icône à côté, /* ellipsis-v */.
padding: 0 0.1ex 0 1ex;, à l'usage ce serait bien d'avoir un 1ex aussi sur la droite pour étendre la zone cliquable, à combiner je dirais avec un margin-right: -0.5ex pour ne pas trop s'éloigner du bord.
Décaler le menu un peu sur la droite, pour qu'une descente verticale du curseur passe à l'intérieur.
Visualiser de manière plus nette le survol des liens du menu; les faire tenir les bords.
De manière générale, relier visuellement le lien/bouton et le menu ouvert; ça demande à revoir le js pour poser une classe lors de l'ouverture.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
Tu dois avoir un commit en local avant celui-là qui fait qu'il ne s'applique pas sur le master.
J'ai testé en rajoutant le code à la main et je ack.
Mis à jour par Frédéric Péters il y a environ 7 ans
Oui j'ai un tas de commits locaux mais ils sont dans des patchs pour lesquels une relecture est bienvenue :) (#7760 et cie)
Mis à jour par Frédéric Péters il y a environ 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 7ebb003124429914c0c204e8403c2a5b50c0b0eb
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Tue Jan 31 11:42:16 2017 +0100
add support for extra actions in 'sandwich' menu (#14830)
The menu should be a list with "extra-actions-menu" class and the
toggler a link with class "extra-actions-menu-opener".
For example:
<a class="extra-actions-menu-opener"></a>
<ul class="extra-actions-menu">
...
</ul>
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
- Statut changé de Résolu (à déployer) à Fermé
add support for extra actions in 'sandwich' menu (#14830)
The menu should be a list with "extra-actions-menu" class and the
toggler a link with class "extra-actions-menu-opener".
For example:
<a class="extra-actions-menu-opener"></a>
<ul class="extra-actions-menu">
...
</ul>