Uh oh!
There was an error while loading. Please reload this page.
Cape Town | 26-ITP-May | Shafiek Walker | Sprint 2 | Wireframe - #1274
Cape Town | 26-ITP-May | Shafiek Walker | Sprint 2 | Wireframe#1274shafiekwalker7861 wants to merge 2 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This comment has been minimized.
This comment has been minimized.
LonMcGregor
left a comment
There was a problem hiding this comment.
Great work, this page is almost complete. Just one small issue with the footer - see my other comment
Uh oh!
There was an error while loading. Please reload this page.
LonMcGregor
commented
May 18, 2026
I see you only made a single commit once the task was complete. It is OK to make smaller commits incrementally as you make changes instead of one big commit. |
shafiekwalker7861
commented
May 18, 2026
Hi LonMcGregor, Thanks for the feedback. Much appreciated. So what do I do next |
LonMcGregor
commented
May 18, 2026
Next steps: If you see my earlier comment, it seems the footer is stuck to the bottom of the page. Try to find a way to make it stick to the bottom of the viewport instead. Then, make a new commit and set the "needs review" label on this PR and I'll take another look. |
LonMcGregor
commented
May 18, 2026
Good work. This task is now complete! |
illicitonion
commented
Aug 24, 2026
Closing PR because the May ITP run has finished. Feel free to re-open if you're still working on it. |

Cape Town | 26-ITP-May | Shafiek Walker | Sprint 2 | Wireframe
Self checklist
Changelist
Created a wireframe webpage using semantic HTML and CSS
Added sections explaining wireframes, Git branches, and README files
Styled the page layout and added images
Practiced Git branching and pull request workflow
Questions
This is my first clean pull request workflow, so any feedback on structure or Git usage would be appreciated.