manche: create theme (#88587) #546

Merged
csechet merged 1 commits from wip/83773-theme-manche into main 2024-04-02 15:57:33 +02:00
csechet added 1 commit 2024-03-19 08:58:16 +01:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
80ab2cd622
manche: create theme (#83773)
Owner

retours code

cf au sein des

retours graphiques.

Voir les captures jointes pour illustration

  • Le header passe moyen en mode conecté avec un nom un peu à rallonge (voir capture)
  • Idem pour les toplinks en mobile, le nom du user touche deconnexion
  • Le titre de la démarche est énorme sur un mobile
  • Le contenu de la page passe par dessus le menu ouvert en mobile.
  • Les widget select2 ne se voient pas, pas de border bottom
**retours code** cf au sein des **retours graphiques.** Voir les captures jointes pour illustration * Le header passe moyen en mode conecté avec un nom un peu à rallonge (voir capture) * Idem pour les toplinks en mobile, le nom du user touche deconnexion * Le titre de la démarche est énorme sur un mobile * Le contenu de la page passe par dessus le menu ouvert en mobile. * Les widget select2 ne se voient pas, pas de border bottom
tjund requested changes 2024-03-19 16:06:50 +01:00
Dismissed
@ -0,0 +93,4 @@
.icon-bar-1 { top: 33% }
.icon-bar-2 { top: 50% }
.icon-bar-3 { top: 68% }
Owner

Pour faire un bouton rond en modifiant l'espace entre le bord du bouton et les barres, regarde du côté de l'ajout simplement d'un border transparent au bouton, c'est beaucoup plus simple. Inutile de repositionner la position des barres.

Pour faire un bouton rond en modifiant l'espace entre le bord du bouton et les barres, regarde du côté de l'ajout simplement d'un border transparent au bouton, c'est beaucoup plus simple. Inutile de repositionner la position des barres.
@ -0,0 +105,4 @@
}
@media($max-mobile-viewport) {
.gru-nav-button.toggled + ul {
height: 100vh;
Owner

peut-être qu'utiliser $responsive-menu: left-to-right; comme base et le transformer un right-to-left (voir thème caen par exemple) sera plus simple et plus safe ?

peut-être qu'utiliser `$responsive-menu: left-to-right;` comme base et le transformer un right-to-left (voir thème caen par exemple) sera plus simple et plus safe ?
@ -0,0 +173,4 @@
}
}
.pk-button-cancel {
Owner

Tu as essayé simplement ?

%cancel-button {
		-- background: $blue-dark;
		-- color: white;
		border: 1px solid white;
}
Tu as essayé simplement ? ``` %cancel-button { -- background: $blue-dark; -- color: white; border: 1px solid white; } ```
Owner

Et si on ajoute une image à une page, c'est pas top.

Et si on ajoute une image à une page, c'est pas top.
csechet force-pushed wip/83773-theme-manche from 80ab2cd622 to ed967fce23 2024-03-25 08:44:43 +01:00 Compare
csechet requested review from tjund 2024-03-25 08:45:09 +01:00
csechet force-pushed wip/83773-theme-manche from ed967fce23 to 5d1a58d828 2024-03-25 08:48:23 +01:00 Compare
csechet changed title from manche: create theme (#83773) to manche: create theme (#88587) 2024-03-25 08:48:41 +01:00
Owner

À 1200px de viewport, les toplinks sont hors viewport et il y a un scroll horizontal

À 1200px de viewport, les toplinks sont hors viewport et il y a un scroll horizontal
tjund requested changes 2024-03-25 17:44:22 +01:00
Dismissed
@ -0,0 +65,4 @@
padding-left: $space-xlarge;
}
@media($max-mobile-viewport) {
Owner

vide, à supprimer

vide, à supprimer
@ -0,0 +103,4 @@
}
+ ul {
width: 100vw;
Owner

ça s'applique en desktop et surement ça qui chasse les toplinks hors viewport

ça s'applique en desktop et surement ça qui chasse les toplinks hors viewport
csechet force-pushed wip/83773-theme-manche from 5d1a58d828 to 1211c18c98 2024-03-29 14:43:42 +01:00 Compare
tjund approved these changes 2024-04-02 11:12:12 +02:00
csechet force-pushed wip/83773-theme-manche from 1211c18c98 to 7316380a38 2024-04-02 15:26:10 +02:00 Compare
csechet merged commit 12027802cc into main 2024-04-02 15:57:33 +02:00
csechet deleted branch wip/83773-theme-manche 2024-04-02 15:57:33 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#546
No description provided.