loire-atlantique-cd44: add theme (#70245) #166

tjund · 2023-02-07T16:21:49+01:00

tjund commented

2023-02-07 16:21:49 +01:00

import a theme made by cd44

Je repris les html pour éviter les surcharges complètes de template.
** Pas mal de surcouches côté pages authentic, j'ai essayé de simplifier.
À peine touché aux CSS
Utilisé nos fonts, remplacé la "blogger" par "Abel" (en faux bold)

import a theme made by cd44 * Je repris les html pour éviter les surcharges complètes de template. ** Pas mal de surcouches côté pages authentic, j'ai essayé de simplifier. * À peine touché aux CSS * Utilisé nos fonts, remplacé la "blogger" par "Abel" (en faux bold)

tjund added 1 commit 2023-02-07 16:21:50 +01:00

gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details

gitea/publik-base-theme/pipeline/head Something is wrong with the build of this commit Details

d9edeb2607 loire-atlantique-cd44: add theme (#70245 )

import a theme made by cd44

aberriot approved these changes 2023-02-09 16:48:16 +01:00

aberriot left a comment

Relecture un peu particulière puisqu'ici on intègre tu code écrit par d'autre.

Testé en local, tout charge bien et le rendu me semble suffisamment proche de https://demarches.loire-atlantique.fr/, à l'exception de cette histoire de police non libre, que tu as déjà expliqué

Je n'ai pas pu accéder à des démarches sur leur site (il faut un compte), donc pour la partie formulaire je ne suis pas certaine de si le rendu est identique mais bon, comme tu dis tu n'as quasiment pas touché à leur CSS, donc il n'y a pas de raison.

Okay pour moi :)

Relecture un peu particulière puisqu'ici on intègre tu code écrit par d'autre. Testé en local, tout charge bien et le rendu me semble suffisamment proche de https://demarches.loire-atlantique.fr/, à l'exception de cette histoire de police non libre, que tu as déjà expliqué Je n'ai pas pu accéder à des démarches sur leur site (il faut un compte), donc pour la partie formulaire je ne suis pas certaine de si le rendu est identique mais bon, comme tu dis tu n'as quasiment pas touché à leur CSS, donc il n'y a pas de raison. Okay pour moi :)

static/loire-atlantique-cd44/extra.js

						
				@ -0,0 +1,81 @@

				$(function() {

aberriot commented

2023-02-09 16:40:57 +01:00

Je découvre au passage qu'un extra.js dans un dossier de thème est chargé automatiquement par combo, sympa !

Je découvre au passage qu'un `extra.js` dans un dossier de thème est chargé automatiquement par combo, sympa !

tjund merged commit 8be5577c1b into main

2023-02-13 11:48:07 +01: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#166

				`@ -0,0 +1,81 @@`
				`$(function() {`