entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 11 Releases Wiki Activity

Armentières : ajustement (#77625) #269

Merged
csechet merged 21 commits from wip/77625-armentieres-ajustements into main 2023-05-31 12:05:32 +02:00
Conversation 1 Commits 21 Files Changed 8 +147 -57
csechet commented 2023-05-16 16:22:34 +02:00
Owner
Copy Link
No description provided.
csechet added 20 commits 2023-05-16 16:22:34 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
6040e72203 armentieres: add link class to add form icon (#77625)
pmarillonnet reviewed 2023-05-30 15:22:36 +02:00
pmarillonnet left a comment
Owner
Copy Link

Rubik cède sa place de famille de police principale pour ce thème au profit de Manrope, mais cette première reste en police de subtitution ici et là, lorsque Viga ou Archivo Black ne sont pas trouvées. C’est volontaire ?

Rubik cède sa place de famille de police principale pour ce thème au profit de Manrope, mais cette première reste en police de subtitution ici et là, lorsque Viga ou Archivo Black ne sont pas trouvées. C’est volontaire ?
pmarillonnet approved these changes 2023-05-30 15:25:45 +02:00
pmarillonnet left a comment
Owner
Copy Link

Je te laisse voir en fonction de mon précédent commentaire (et si effectivement on retire complètement l’usage de Rubik, on peut en retirer l’import dans style.scss). Pour le reste c’est des petits ajustements cosmétiques, pas envie de checkout en local pour comparer un à un les points relevés par Armentières et cocher une case à chaque fois. Ack un peu complaisant.

Je te laisse voir en fonction de mon précédent commentaire (et si effectivement on retire complètement l’usage de Rubik, on peut en retirer l’import dans `style.scss`). Pour le reste c’est des petits ajustements cosmétiques, pas envie de checkout en local pour comparer un à un les points relevés par Armentières et cocher une case à chaque fois. Ack un peu complaisant.
csechet added 1 commit 2023-05-30 15:32:47 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
5cf7a16201 armentieres: remove references to Rubik font (#77625)
csechet commented 2023-05-30 15:35:06 +02:00
Author
Owner
Copy Link

Je te laisse voir en fonction de mon précédent commentaire (et si effectivement on retire complètement l’usage de Rubik, on peut en retirer l’import dans style.scss). Pour le reste c’est des petits ajustements cosmétiques, pas envie de checkout en local pour comparer un à un les points relevés par Armentières et cocher une case à chaque fois. Ack un peu complaisant.

Bien vu. En effet, il y a eu des allers-retours sur les fonts à utiliser et Manrope est sortie de leur chapeau pendant les derniers retours, je n'avais pas fait attention au fait que Rubik n'était plus utilisée dans les faits. J'ai retiré les références à la fonte, merci Paul.

> Je te laisse voir en fonction de mon précédent commentaire (et si effectivement on retire complètement l’usage de Rubik, on peut en retirer l’import dans `style.scss`). Pour le reste c’est des petits ajustements cosmétiques, pas envie de checkout en local pour comparer un à un les points relevés par Armentières et cocher une case à chaque fois. Ack un peu complaisant. Bien vu. En effet, il y a eu des allers-retours sur les fonts à utiliser et Manrope est sortie de leur chapeau pendant les derniers retours, je n'avais pas fait attention au fait que Rubik n'était plus utilisée dans les faits. J'ai retiré les références à la fonte, merci Paul.
csechet merged commit a425f367a6 into main 2023-05-31 12:05:32 +02:00
csechet deleted branch wip/77625-armentieres-ajustements 2023-05-31 12:05:32 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#269
No description provided.
Delete Branch "wip/77625-armentieres-ajustements"

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?