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 8 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
Sambuxc
left a comment
There was a problem hiding this comment.
Please check that all fields have the correct mandatory validation
added required field to the size selector
|
Congrats you're PR is completed :) |


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