Skip to content

Conversation

ramseycoder
Copy link
Collaborator

No description provided.

@linear
Copy link

linear bot commented Oct 18, 2021

NAN-20 Page d'exercise

Gestion de l'affichage d'un exercise:

  • rendu du sujet
  • affichage des proprieter de l'exercise (titre, notions, etc…)
  • affichage du status de progression (exercise passer)

pas besoin de gerer l'editeur / execution de code ici

View original card in Trello


Kouamé Rameaux Koffi 2021-10-14

#82
build : https://9f307af3.platform-nan-dev-8sl.pages.dev/exercise

Kouamé Rameaux Koffi 2021-10-08

pull request : nan-ci/platform: Pull Request 69
lien build : https://3d1c1209.platform-nan-dev-8sl.pages.dev/student/exercise

@ramseycoder ramseycoder self-assigned this Oct 18, 2021
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 18, 2021

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: ba28c4f
Status: ✅  Deploy successful!
Preview URL: https://3385c0c0.platform-nan-dev-8sl.pages.dev

View logs

@ramseycoder ramseycoder force-pushed the ramsey/nan-20-page-dexercise branch 3 times, most recently from 0052ce2 to e310d9f Compare October 25, 2021 12:37
@ramseycoder ramseycoder marked this pull request as ready for review October 25, 2021 15:26
Copy link
Member

@kigiri kigiri left a comment

Choose a reason for hiding this comment

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

cool franchement on y est presque

class={`mli ${link ? 'link' : ''}`}
onClick={(e) => {
if (link) navigate(link)
}}
Copy link
Member

Choose a reason for hiding this comment

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

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>
)
}
Copy link
Member

Choose a reason for hiding this comment

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

pas besoin du return {} ici, ca meriterais peu etre une classe au lieu du style direct ?


.exercise h3 {
font-weight: bolder;
}
Copy link
Member

Choose a reason for hiding this comment

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

N'hesite pas a rassembler les declaration si elle sont comune

    .exercise h1,
    .exercise h3 {
      font-weight: bolder;
    }

}
`)
export const Exercise = () => {
return (
Copy link
Member

Choose a reason for hiding this comment

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

pareil ici utilise le return impllicite, ca reduit l'indentation d'un niveau

@ramseycoder ramseycoder force-pushed the ramsey/nan-20-page-dexercise branch from 03af243 to ba28c4f Compare November 2, 2021 11:34
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