NW-6 | AREEB-SATTAR | HTML-CSS | [TECH ED] Cakes Co | WEEK-2

[areebsattar]

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[bot]]

✅ Deploy Preview for cyf-module-html-css ready!

Name	Link
🔨 Latest commit	301da67dca984b97cebcd8e358d0e7cfee9f3ce3
🔍 Latest deploy log	https://app.netlify.com/sites/cyf-module-html-css/deploys/652b00f5caba3600080fadd4
😎 Deploy Preview	https://deploy-preview-143--cyf-module-html-css.netlify.app/cakes-co
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: Accessibility: Best Practices: SEO: PWA: - View the detailed breakdown and full score reports

---

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

[JayMayer]

Great work Areeb! I can clearly see you have used a mobile-first approach to the design, and the site keeps all the relevant content visible on mobile, tablet, and desktop. Your images don't become distorted or squashed, they look delicious whatever the screen size! I also like how the menu bar continues to shift with the screen size on desktop to add some flair 😀

And fantastic work getting the hamburger menu to function, there's some great CSS selectors in there!

Aside from a few inline comments on code styling, I only want to comment on some places where the layout might need to be considered more closely. It's only a minor thing, but can sometimes make a site look less polished. For example, in the image below, the grid could be centred on the screen, with the top image aligning to the bottom images

Keep up the great work!