koeenm/the-web-is-for-everyone-interactive-functionality

Ontwerp en maak voor een opdrachtgever een interactieve toepassing die voor iedereen toegankelijk is

the-web-is-for-everyone-interactive-functionality's Issues

1 Dialog met meerdere fieldsets

Je gebruikt nu meerdere dialogs, zorg dat je 1 dialog aanmaakt met meerdere fieldsets.

Sluitknop voor de dialog

Misschien is het beter om een sluitknop toe te voegen in de dialog, zodat de gebruiker weet dat hij deze kan sluiten.

Feedback pour moi

vormgeving

• Vormgeving ziet er goed uit. kan op sommige plekken alleen nog wat padding gebruiken.

• Heel vet! formuliertje? even traantje leggen zo goed.
• verschillende pagina's aan de praat gekregen lekker bezig!
• Responsive? bravo

Code

server.js

• Is helemaal goed te begrijpen ook erg overzichtelijk

index.ejs

the-web-is-for-everyone-interactive-functionality/views/index.ejs

Lines 4 to 8 in 0217c67

	<section class="explanation">
	<p>Op deze buurtwebsite kunnen bewoners van Amsterdam West informatie met elkaar delen.
	Hier vind je alle activiteiten die er in de buurt zijn, informatie over initiatieven, kan je een oproep plaatsen en jouw eigen dienstenaanbod bekend maken.</p>
	</section>

• section? Een section begint met een heading, je zou moeten kijken of je dit element wilt vervangen met een div of een article

faq.ejs

the-web-is-for-everyone-interactive-functionality/views/faq.ejs

Lines 17 to 23 in 0217c67

	<style>
	h2 {
	color: var(--colorOne);
	margin-left: 1em;
	}

• Waarom maak je hier gebruik van <style>? Hiervoor heb je het style.css file voor

style.css

• @font-face moet altijd bovenaan. Dit is het allereerste wat je wilt importeren wanneer je een pagina opent.

the-web-is-for-everyone-interactive-functionality/public/style.css

Lines 15 to 17 in 0217c67

	@font-face {
	font-family: raleway;
	src: url(/fonts/Raleway-VariableFont_wght.ttf);

• Je doet veel met css nesting, lekker bezig. Het enige wat ik nog zou meegeven is om eerste de basis elementen te stijlen. Je hebt bijvoorbeeld nu een stijlen voor een h1 in een header, maar hoe ziet een h1 op zichzelf eruit? Ik zou dat de volgende keer eerst stijlen, waarna je dus binnen die stijlen kan werken. Maar dan heb je teminste een goede basis. In mijn css doe ik het ook al een beetje, maar zo zou je het misschien de volgende keer kunnen aanpakken. More DRY :)
  

  the-web-is-for-everyone-interactive-functionality/public/style.css

  Lines 38 to 51 in 0217c67

  	header {
  	display: flex;
  	padding: 0.5em;
  	background-color: var(--colorTwo);
  	ul {
  	display: none;
  	}
  	h1 {
  	margin: 0 auto;
  	color: var(--colorOne);
  	}

• Misschien de @media op een plek zetten? Zet ze anders helemaal beneden, teminste als je een algemene verdeling wilt maken ervan. Maar dit is ook prima.

waar bevindt de gebruiker zich?

Het is nu nog niet duidelijk waar de gebruiker zich bevindt. Je kunt het aantal stappen van een proces weergeven. En ook aangeven op welke pagina de gebruiker zich bevindt, bijv. aanbod of bij vraag.