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
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. |
Hi LonMcGregor, Thanks for the feedback. Much appreciated. So what do I do next |
|
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. |
|
Good work. This task is now complete! |

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.