Feedback JS Opdracht 1: Lightopia
@Work 1
Algemeen
Feedback
Volg folder structuur
- Juiste folder naam
- Juiste folder structuur
- Screencast gemaakt
Kijk robots.txt na
Jouw robots.txt moet ingesteld staan zodat bots geen toegang krijgen
User-agent: *
Disallow: /Afspraken JavaScript
Feedback
Algemeen
- Gebruik const en let, geen var
- Voor naamgeving gebruik je camelCase
- Gebruik Engelstalige namen
Inline JS
Inline JS in HTML is verboden
FOUT:
Script linken
Zet je script onderaan de body ipv DOMContentLoaded
FOUT:
document.addEventListener("DOMContentLoaded", (event) => {
console.log("DOM fully loaded and parsed");
});Script linken
Zet je script onderaan de body ipv DOMContentLoaded
JUIST:
// DOM is sowieso geladen
console.log("DOM fully loaded and parsed");
<body>
...
...
<script src="main.js"></script>
</body>Gebruik classes
Gebruik steeds classes tenzij je niet anders kan, vermijd aanpassen van style property
FOUT:
JUIST:
$element.style.display = "grid";$element.classList.add("visible").visible {
display: grid;
}Beperk class aanpassingen
Wijzig zo weinig mogelijk classes om één doel te bereiken!
FOUT:
Beperk class aanpassingen
Wijzig zo weinig mogelijk classes om één doel te bereiken!
FOUT:
$questionContainer.classList.add("visible");
$answer.classList.add("visible");Beperk class aanpassingen
Gebruik het parent element en maak gebruik van conditionele styling
JUIST:
$questionContainer.classList.add("visible");.question {
height: 5rem;
}
.question .answer {
display: none;
}
.question.visible {
height: auto;
}
.question.visible .answer {
display: block;
}Als laatste...
Feedback
Gebruik de leerstof
- Tijdens de lessen krijg je al heel wat tips / aanzet, gebruik dit!
- Er zijn studenten in geslaagd om zelf de gekregen JS niet in te dienen, dit kan echt niet
Gebruik console.log
- Het is onmogelijk om code te schrijven zonder af en toe te controleren wat de waarde van een variabele is (en dat het dus is wat je verwacht)
@Work1 - Feedback JS Opdracht 1: Lightopia
By Lecturer GDM
@Work1 - Feedback JS Opdracht 1: Lightopia
Feedback op Opdracht 1 Lightopia (2023-2024)
