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

feat(content) add and use LineLink inline component #194

Merged
merged 4 commits into from
Nov 7, 2023

Conversation

Delapouite
Copy link
Contributor

Bonjour

De nombreuses pages décrivant les voies lyonnaises mentionnent des intersection avec d'autres voies.

Ce commit introduit un nouveau composant LineLink qui peut être utilisé inline dans le contenu des markdowns afin d'obtenir des liens de la couleur de la ligne ciblée.

Avant:

[Voie Lyonnaise 8](https://cyclopolis.fr/voie-lyonnaise-8) du boulevard du Valvert à l'avenue Victor Hugo

image

Après:

:line-link{line=8} du boulevard du Valvert à l'avenue Victor Hugo

image

Copy link
Owner

@benoitdemaegdt benoitdemaegdt left a comment

Choose a reason for hiding this comment

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

Hyper stylé, c'est nettement plus visuel comme ça 👍

J'ai fait 1 ou 2 modifs direct sur la PR. Si ça te va, on peut merger ça demain !

@@ -0,0 +1,21 @@
<template>
<NuxtLink :to="`/voie-lyonnaise-${line}`" :style="`color: ${color}; text-decoration-color: ${color};`">
Copy link
Owner

@benoitdemaegdt benoitdemaegdt Nov 6, 2023

Choose a reason for hiding this comment

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

j'ai remplacé la balise <a> par <NuxtLink>. J'essaye d'utiliser ça pour ce qui est lien interne, ça aide Nuxt à s'y retrouver quand il build ou crawl.

line: { type: String, required: true }
});

const color = getLineColor(Number(props.line));
Copy link
Owner

Choose a reason for hiding this comment

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

j'ai ajouté une composable qui permet de récupérer la couleur d'une ligne à partir de son numéro.
(composables/useColors.ts). ça évite d'utiliser Nuxt Content juste pour récupérer la couleur de la ligne :)

@Delapouite
Copy link
Contributor Author

Top ! Merci pour les ajustements propres à Nuxt (framework que je découvre via ce projet).

Aussi je vois que les semicolons font leur apparition ce qui confirme que les warnings de ESLint que j'avais choisi d'ignorer pour etre raccord avec les autres fichiers que j'avais pu lire étaient bien réels. 👍

Content que ça puisse etre mergé, dès que tu en as l'occasion.

@benoitdemaegdt
Copy link
Owner

Aussi je vois que les semicolons font leur apparition ce qui confirme que les warnings de ESLint que j'avais choisi d'ignorer pour etre raccord avec les autres fichiers que j'avais pu lire étaient bien réels. 👍

Et oui en effet, je commence à ajouter un peu de rigueur sur les règles eslint et le typage TS, mais jusqu'ici c'était très souple :D

Copy link
Owner

@benoitdemaegdt benoitdemaegdt left a comment

Choose a reason for hiding this comment

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

Merci pour cette belle contribution ! 👍

@benoitdemaegdt benoitdemaegdt merged commit 15dad71 into benoitdemaegdt:main Nov 7, 2023
@Delapouite Delapouite deleted the feat/line-link branch November 7, 2023 08:13
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.

2 participants