Manchester | 26 ITP May | Szidi Bodo | Sprint 1 | Form Control#1259
Manchester | 26 ITP May | Szidi Bodo | Sprint 1 | Form Control#1259bodoszidi wants to merge 7 commits into
Conversation
-removed text from the form section -updated footer with my name
added name and email with validation
added T-shirt colour and size to the form with validation
added css style to the content
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This comment has been minimized.
This comment has been minimized.
|
Great start a few points to mention:
|
Question ResponseQ. Had difficulties with centring the content - I'm not quite sure why flex on my CSS was not working What's important to understand with flexbox is depending on the flex-direction value (row/column), will change how justify-content and align-items work. It's useful to play around with flex row and adjust the values of the other flex properties vs. flex column - compare how they work. A useful tool I have used in my early days of learning flexbox - https://flexbox.malven.co/ Pay attention to the keywords "main axis" and "cross axis". Have a read up and try to achieve what you were intending on this PR to make your form content central. Just to show you it can be done, I have achieved centering the form content using only 3 css properties:
|
closed the select tag and added 2 buttons to the form
-updated form field and add required attributes -fixed styling by adding flex and updating width on CSS
updated the button css style fixed the colour selector css


Self checklist
Changelist
Created the form with validation and basic style - only using HTML and CSS
Lighthouse accessibility check is 100%
Questions
Had difficulties with centring the content - I'm not quite sure why flex on my CSS was not working