Birmingham | ITP-May-26 | Toluwalase Tiamiyu | Sprint 1 | Form Control

[TTiamiyu]

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

In this commit i have created a form that allows users to select size, color and also has a submit button,

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	4819813
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a268144dd46160008d58010
😎 Deploy Preview	https://deploy-preview-1416--cyf-onboarding-module.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	2 paths audited Performance: 100 (no change from production) Accessibility: 100 (no change from production) Best Practices: 100 (no change from production) SEO: 91 (🟢 up 5 from production) PWA: - View the detailed breakdown and full score reports
🤖 Make changes	Run an agent on this branch

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[cjyuan]

@TTiamiyu Since this is a continuation of your previous PR (PR#1257) and not a new PR

• It may be helpful to note the current status of the PR in the description to prevent other reviewers from assuming it is a new submission.
• You could tag the original reviewer in a comment and invite them to continue reviewing this PR.

[TTiamiyu]

The initial reviewer was tagged in the previous PR before i closed it.

[TTiamiyu]

Should I tag the initial reviewer again in the present PR, even if I already tagged in the old PR with the link to this new PR?

[Luro91]

I see an accessibility score of 92 in Lighthouse. How can you get it to 100?

[Luro91]

Should I tag the initial reviewer again in the present PR, even if I already tagged in the old PR with the link to this new PR?

I found it. As mentioned when you write a short explanation in the PR descriptions it's easier to see as mention link is a bit hidden between the other comments

[TTiamiyu]

@Luro91, the last couple of commits has addressed the following issues raised

• The aria attribute was added to the html to improve the experience users that use screen readers have
• A css file was added to improve the page style and also address the lighthouse rating.

[Luro91]

Well done the code works as expected

[illicitonion]

Closing PR because the May ITP run has finished. Feel free to re-open if you're still working on it.