-
Notifications
You must be signed in to change notification settings - Fork 24
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
Conversation
There was a problem hiding this 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};`"> |
There was a problem hiding this comment.
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)); |
There was a problem hiding this comment.
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 :)
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. |
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 |
There was a problem hiding this 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 ! 👍
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:
Après: