Conversation
NAN-20 Page d'exercise
Gestion de l'affichage d'un exercise:
Kouamé Rameaux Koffi 2021-10-14 #82 Kouamé Rameaux Koffi 2021-10-08 pull request : nan-ci/platform: Pull Request 69 |
Deploying with
|
| Latest commit: |
ba28c4f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3385c0c0.platform-nan-dev-8sl.pages.dev |
ramseycoder
force-pushed
the
ramsey/nan-20-page-dexercise
branch
3 times, most recently
from
0052ce2 to
e310d9f
Compare
kigiri
left a comment
kigiri
left a comment
There was a problem hiding this comment.
cool franchement on y est presque
component/markdown.jsx
Outdated
| class={`mli ${link ? 'link' : ''}`} | ||
| onClick={(e) => { | ||
| if (link) navigate(link) | ||
| }} |
There was a problem hiding this comment.
Il faut pas navigate sur un onclick, a eviter, pour gerer ca utilise le compent Link de elements.jsx
ca va creer un vrais a avec la bonne proprieter HREF, les avantages:
- si on passe la souris dessus on peu voir l'addresse
- si on ctrl + click ou middle click ca ouvre dans un nouvel onglet
- on a le menu click droit special des liens
- on a pas besoin de css special, il a deja les style d'un lien (pointer)
- les outils de liseuse pour les personnes qui on des probleme de vision comprene que c'est un lien et peuvent le gerer
| {children} | ||
| </Div> | ||
| ) | ||
| } |
There was a problem hiding this comment.
pas besoin du return {} ici, ca meriterais peu etre une classe au lieu du style direct ?
page/exercise.jsx
Outdated
|
|
||
| .exercise h3 { | ||
| font-weight: bolder; | ||
| } |
There was a problem hiding this comment.
N'hesite pas a rassembler les declaration si elle sont comune
.exercise h1,
.exercise h3 {
font-weight: bolder;
}| } | ||
| `) | ||
| export const Exercise = () => { | ||
| return ( |
There was a problem hiding this comment.
pareil ici utilise le return impllicite, ca reduit l'indentation d'un niveau
ramseycoder
force-pushed
the
ramsey/nan-20-page-dexercise
branch
from
03af243 to
ba28c4f
Compare
2 participants
No description provided.