Skip to content

Comments

Brussels | itp-2026-1 | Sarah Almadi | Week 2 |Feature/form controls#84

Open
sarahalmadi wants to merge 6 commits intoHackYourFutureBelgium:mainfrom
sarahalmadi:feature/form-controls
Open

Brussels | itp-2026-1 | Sarah Almadi | Week 2 |Feature/form controls#84
sarahalmadi wants to merge 6 commits intoHackYourFutureBelgium:mainfrom
sarahalmadi:feature/form-controls

Conversation

@sarahalmadi
Copy link

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title

  • My changes meet the requirements of the task

  • I have tested my changes

  • My changes follow the style guide

Changelist

Used Semantic HTML based on MDN
Added inputs to my labels
Used 'Radio buttons' and option selection for colors and sizes
Added a footer with my name
checked the code validation

Questions

Ask any questions you have for your reviewer.

@sarahalmadi sarahalmadi marked this pull request as ready for review February 23, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant