Melvynx / beginreact-fundamentals

3 stars 8 forks source link

Unnecessary comment for 'src/exercise/1-no-react/01.html' #2

Open kml93 opened 3 months ago

kml93 commented 3 months ago

In your code, they are unnecessary comment. Who have no logic to be there.

<!doctype html>
<html lang="en">
  {{> header}}
  <body>
    {{> header-bar }}

    <!-- 🦁 crée une div avec l'id "root" et la classe "container" -->
    <!-- ℹ️ presque toutes les app React ont une "root".
          C'est le premier nœud de notre arbre. -->
  </body>

  <script>
    // 🦁 crée un div avec l'id "root" et la classe "container"

    // 🦁 crée un button

    // 🦁 ajoute le text "Hello 👋" dedans

    // 🦁 ajoute le button dans l'élément "root"

    // 🦁 récupère l'élément "root"

    // 🦁 ajoute la div dans l'élément "root"
  </script>
</html>

We should keep only these comments :

<!-- 🦁 crée une div avec l'id "root" et la classe "container" -->
<!-- ℹ️ presque toutes les app React ont une "root".
          C'est le premier nœud de notre arbre. -->
<script>
// 🦁 crée un button

// 🦁 ajoute les classes "btn" et "btn-primary" dedans

// 🦁 ajoute le text "Hello 👋" dedans

// 🦁 récupère l'élément "root"

// 🦁 ajoute le button dans l'élément "root"
</script>

or these comments to keep things logical :

<script>
// 🦁 crée un div avec l'id "root" et la classe "container"

// 🦁 crée un button

// 🦁 ajoute les classes "btn" et "btn-primary" dedans

// 🦁 ajoute le text "Hello 👋" dedans

// 🦁 ajoute le button dans l'élément "root"
</script>