Je code ziet er netjes uit, maar op de website zelf:

Probeer de meer info tekst in het midden te zetten. Of maak de box zelf kleiner.
Het is een kleine detail maar ik keek er telkens naar terug.

Misschien is het handiger om namen met betekenis te geven.

Je sluit hier eerst een div af en daarna de article, terwijl die article in de div staat. Of kijk ik verkeerd?

Bij je content gaat er nog iets mis met de breedte. Ik zie dat de pagina voor 75% breed is. Daarna zie ik niks meer. (Zie afbeelding hieronder)

Kijk even of je width en height kloppen van de site.

Je lijkt hier een element te gebruiken dat niet in html mag

Volgens mij doet dit element nu ook niks dus kan je gewoon weghalen

bron:

Bij sommige buttons heb je gebruik gemaakt van feedforward door een hover toe te voegen, maar dit heb je niet overal gedaan. Misschien handig om ook bij de andere buttons toe te voegen :)

Je gebruik dezelfde id meerdere keren.

Je mag een ID maar 1 keer gebruiken, je kan dit oplossen door verschillende namen te gebruiken.

bron: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/id

Ik begrijp de structuur van de html, en het is duidelijk waar verschillende onderdelen van de site zijn door gebruik van articles.

Het enige echt aan te merken is dat je van een h2 naar een h4 overspringt, terwijl je beter in plaats van die h4 een h3 kan maken, voor de volgorde van de website

Tip:
Ik zie in de HTML van je app-landingspagina dat je op verschillende manieren elementen nest, en laat inspringen. Neem bijvoorbeeld deze eerste unordered-list. Hier zet je de link direct binnen in de list-item, op dezelfde regel. Terwijl je bij een andere unordered-list (de tweede) hier enters voor gebruikt. Waardoor je in plaats van 1 regel, 3 regels gebruikt. Ik denk dat het beter zou zijn als je consistent dezelfde manier toepast voor je gehele pagina. Zo behoud je meer overzicht over wat er allemaal in je pagina staat.

Top:
Verder vind ik dat je HTML mooi geschreven is. Je maakt gebruik van de juiste elementen, die voor de juiste functie dienen. En je zet overal mooie comments bij zodat het voor iedere lezer duidelijk is waar de code voor bedoeld is.

Je mag geen value attribuut toevoegen aan het meta element in de site.

Je mag niet een icon attribuut gebruiken binnen een link. Misschien kan je een oplossing hiervoor vinden?

Je kan deze functie vervangen door in je css de :hover atribuut te gebruiken. Hierdoor heb je een heel stuk JS code niet nodig.

Meer informatie is hier te vinden:
https://developer.mozilla.org/en-US/docs/Web/CSS/:hover

Ik begrijp hoe de css in elkaar zit en de namen zijn duidelijk aangegeven vind ik, je kan wel bedenken welke classes bij welke onderdelen van de site horen, door het gebruik van de naam.

Ik zou alleen kijken of je herhaling kan vermijden, door bijvoorbeeld te nesten. Nu schrijf je de hele class naam opnieuw op en schrijf je daarachter een 'p' of een 'a' om zo deze te stylen. Maar je kan dit ook nesten waardoor je, alles makkelijk tegelijkertijd kan sluiten, en je niet telkens de classname hoeft te herhalen. Maar dat doe ik zelf dus dat is ook eigenlijk meer een persoonlijke voorkeur