Glasgow | 26-ITP-May | Sandani Kannangara | Sprint 2 | Feature/wireframe

[SandzSoft]

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

• Built a semantic HTML page using header, main, article, and footer elements
• Added three articles covering README files, wireframes, and Git branches
• Included images, headings, summaries, and external links for each article
• Styled the page using CSS Grid and basic layout techniques
• Ensured accessibility best practices (alt attributes and semantic structure)
• Fixed footer to stay at the bottom of the page

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	780e855
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a16bd2d5761f000083bac82
😎 Deploy Preview	https://deploy-preview-1327--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: 86 (no change 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.

[LonMcGregor]

Good work, this meets all the criteria.

There is one place you could improve a bit further - Have a look at your footer. It is correctly fixed to the viewport, but as you scroll, do you see how it overlaps? Can that be fixed to ensure it remains readable?

[SandzSoft]

Good work, this meets all the criteria.

There is one place you could improve a bit further - Have a look at your footer. It is correctly fixed to the viewport, but as you scroll, do you see how it overlaps? Can that be fixed to ensure it remains readable?

Thank you for the feedback. I understand the issue now — because the footer is fixed to the viewport, the page content can scroll underneath it and become partially hidden. I can fix this by adding bottom padding or margin to the main content so there is enough space for the fixed footer and everything remains readable while scrolling.

[LonMcGregor]

Good suggestion. But consider also the readability as you scroll. For example, when this happens:

Could you try making some changes to the CSS to resolve this?

[SandzSoft]

Good suggestion. But consider also the readability as you scroll. For example, when this happens:

Could you try making some changes to the CSS to resolve this?

Thank you for the feedback. I understand the issue now — the fixed footer was overlapping the content while scrolling. I have changed the footer to a static position so it no longer covers the page content and remains readable.

[LonMcGregor]

So, the readme requires the footer be fixed to the viewport, you had this working correctly already.

But when it overlaps, the test of the footer isn't readable.

There is a change you could make that would ensure readability while keeping the footer fixed and visible always. (hint: could you do anything about the background of the footer?)

[SandzSoft]

So, the readme requires the footer be fixed to the viewport, you had this working correctly already.

But when it overlaps, the test of the footer isn't readable.

There is a change you could make that would ensure readability while keeping the footer fixed and visible always. (hint: could you do anything about the background of the footer?)

Thank you for the feedback. I understand now — the footer needs to remain fixed to the viewport as required. I have added a background colour to the footer so that it stays readable even when content scrolls behind it, preventing any overlap issues with the text.

[LonMcGregor]

Great work, this task is complete now