Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

static: add CDC logo #79

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

ludovicdmt
Copy link
Member

No description provided.

Copy link
Contributor

@Marc-AntoineA Marc-AntoineA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Attention au nom de la branche, là tu as utilisé un accent

@@ -139,6 +139,25 @@ hr.full-width {
height: auto; /* Conserve les proportions */
}

/* Bandeau CDC */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce serait mieux que tu remettes ces zones de textes lignes 311 et suivantes, cela permet de mieux voir les différences.

Là ce n’est pas faisable de relire facilement les modifications effectuées.

<div class="text">
{{ with .Resources.GetMatch "telescoop.md" }} {{ .Content }} {{ end }}
</div>
<section class="content">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Semi détail) Je l’avais pas indiqué dans la précédente MR sur le site static, mais là encore on a des différences uniquement de tabulation.

Est-ce que le linter est bien configuré ? 

(cela pourrit bien l’historique git car c’est compliqué de voir quelles modifs ont été réellement effectuées)

</section>

<section class="bandeau-cdc">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En fait, t’as ajouté seulement ça non ? 

@@ -130,7 +130,7 @@ hr.full-width {
}

.bandeau-logo {
flex: 1; /* 1/3 pour le logo */
flex: 0.3; /* 1/3 pour le logo */
Copy link

@QuentinMadura QuentinMadura Feb 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'enleverai probablement les commentaires qui sont présents sur les balises CSS :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ETQ utilisateur.ice je veux un bandeau décrivant la CDC en pied de page
3 participants