tristankatte/informaat-opdracht

Ontwerp en maak een responsive website voor een startup.

informaat-opdracht's Issues

2 classes ipv 1?

` HTML

` hier staan 2 classes die bij elkaar kunnen komen als 1. Dat maakt het een stukje overzichtelijker.

Je sluit <i> tag meerdere keren af

https://github.com/TristanKatte/the-startup-responsive-interactieve-website/blob/71c601cd8035d29c3de1f01ea7e8fdc33e97cd20/index.html#L29

Op lijn 29 begin je de tag goed maar je sluit hem vervolgens 3 keer af. Dit is maar 1 keer nodig.

Dit kan je oplossen door de overige 2 te verwijderen.

Je kan meer lezen over opening en closing tags via: https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Getting_started

Gebruiksrechten

Let op dat de afbeeldingen en huisistijl van Informaat niet under het Open Source licentie valt.
Informaat verleend, eenmalig, voor deze opdracht, het gebruik van haar intellectuele eigendom.

Spatie kan niet gebruikt worden hier. Validator zegt 'illegal character'.

<img src="assets/Post PreviewTest.jpg" alt="header afbeelding">↩

https://html.spec.whatwg.org/multipage/embedded-content.html#the-img-element

Wcag font size

Probeer een minimum font size aan te houden van 16px, voor wcag toegankelijkheid

<Section> hebben geen headings

Informaat-opdracht/index.html

Lines 125 to 131 in b70a721

	<section class="sliderTitleContainer">
	<section class="sliderTitle">
	<p>Voorbeeld project / van meeuwen</p>
	<h2>Slimmer smeertechnisch<br> onderhoud.</h2>
	</section>
	<img class="vanMeeuwenImg" src="assets/logoVanMeeuwen.png" alt="van Meeuwen logo">
	</section>

Ik zag in de w3 validator dat je section geen headings hebben en dit geeft daarom een error

Ik denk dat je dit kunt oplossen door een heading toe te voegen, als de section geen heading nodig heeft kun je die leeg doen en dan hidden.
Of door de section te veranderen naar div bijvoorbeeld.

Goed semantisch

Bijna geen divs en mooie semantische elementen gebruikt!

Font tip

https://www.cdnfonts.com/gilroy-bold.font

Je kunt deze font gebruiken!

Je zou classnames eventueel op een andere manier kunnen creeeren.

https://github.com/TristanKatte/the-startup-responsive-interactieve-website/blob/71c601cd8035d29c3de1f01ea7e8fdc33e97cd20/index.html#L19

Om woorden uit elkaar te houden zou je hoofdletters en streepjes kunnen gebruiken. Ook is BEM (box, element, modification) misschien een handige manier om zo class naming te verbeteren. (dit is vooral handig al ga je in echt grote projecten werken met eventueel buitenlandse klanten zodat zij de code ook beter begrijpen)

Dit is een website met BEM class naming methode. Dit kan misschien wel handig zijn.

niks in de nav?

<nav></nav>
er staat niks in je nav. Deze kan je weghalen of opvullen.