London|Phone Naing|Module-User-Focus-Data-Form|Week2

[Mgphone]

Learners, PR Template

Self checklist

• [✅ ] I have committed my files one by one, on purpose, and for a reason
• [ ✅] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
• [ ✅] I have tested my changes
• [ ✅] My changes follow the style guide
• [✅ ] My changes meet the requirements of this task

Changelist

Briefly explain your PR.

Questions

Ask any questions you have for your reviewer.

[netlify]

✅ Deploy Preview for cyf-ufd ready!

Name	Link
🔨 Latest commit	e3cf7af
🔍 Latest deploy log	https://app.netlify.com/sites/cyf-ufd/deploys/672b63cf039a4200088d102d
😎 Deploy Preview	https://deploy-preview-307--cyf-ufd.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[mrvicthor]

Hi @Mgphone ,
Excellent work here. I have left a few comments for you to address in your CSS file.
Happy to approve once this is resolved.

[mrvicthor]

Hi @Mgphone ,
Fantastic work here with the form.

[mrvicthor]

Hi @Mgphone ,
May I suggest that you use rem instead of px for other css properties. It is advisable to use px for elements with fixed size (for example, border, and box-shadow).

Thanks

[Mgphone]

Hi @mrvicthor
I have just changed all to rem :)

Thanks.

[mrvicthor]

Hi @Mgphone ,
Looks good to me.
Happy to approve.