kosterm14/look-and-feel-corporate-identity

Ontwerp en maak voor een opdrachtgever een component of website op basis van een bestaande huisstijl

Visual thinking home page redesign

Ik moest van de opdrachtgever de home page veranderen zodat deze beter past bij de huisstijl en wat zij in gedachte had.

Beschrijving

De nieuwe home page is hieronder te zien

Kenmerken

Ik heb gebruik gemaakt van de volgende talen:

• HTML
• CSS
• Javascript

Licentie

This work is licensed under GNU GPLv3.

look-and-feel-corporate-identity's Issues

JavaScript ✨

Hoi Marcoooo!
Leuk dat het gelukt is, wel lastig responsive maken met position absolute he? Misschien van de week nog eens naar kijken om het zonder absolute te doen?

Hier mijn JavaScript feedback:

look-and-feel-corporate-identity/scripts/script.js

Line 2 in 18dba3c

var element = document.getElementById("myDIV");

Gebruik omschrijvende variabelen namen; dus geen element maar researchBlock oid... dan is je code een stuk leesbaarder. Dit geldt ook voor je ids.

look-and-feel-corporate-identity/scripts/script.js

Line 6 in 18dba3c

element.classList.toggle("show2");

In principe kan je voor al deze elementen gewoon show als class adden ipv show met een nummer; en deze met css selecteren als #myDIV2.show. Maaaar eigenlijk nog beter; je kan ook een show class op een hogere container (bijvoorbeeld grid-container of body) zetten waardoor je niet 10 classes hoeft toe te voegen maar 1. Dan kan je in je css zo selecteren: .show #myDIV2 {}, snappie?