West Midlands | May 2026 | Muhammad-Burhan Mustafa | Sprint 2 | Wireframe#1379
West Midlands | May 2026 | Muhammad-Burhan Mustafa | Sprint 2 | Wireframe#1379burhan-mustafa wants to merge 4 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
burhan-mustafa
added
Needs Review
cjyuan
left a comment
cjyuan
left a comment
There was a problem hiding this comment.
Code is free of error, and page layout matches the wireframe well. Good job.
One of the acceptance criteria, "The page footer is fixed to the bottom of the viewport", has not yet been satisfied. Can you make the necessary change? (Suggestion: Ask AI what that requirement means).
| <img src="./images/wireframes.png" alt=""> | ||
| <h2>What is the purpose of a Wireframe?</h2> | ||
| <p> | ||
| A wireframe is used as a structural blueprint that allows a developer to envision how a project will look. |
There was a problem hiding this comment.
Supposedly this paragraph is what is expected in the "article summary".
No change needed.
|
|
||
| article p { | ||
| margin-top:4px; | ||
| } |
There was a problem hiding this comment.
It seems the code is not formatted by an automated tool.
Can you follow this guide to enable VS Code's
"Format on Save" option or to use its "Format Document" command to format your HTML and CSS code?
Note: I meant to select the codes on lines 107-113.
| <article> | ||
| <img src="placeholder.svg" alt="" /> | ||
| <h2>Title</h2> | ||
| <img src="./images/branch.png" alt=""> |
There was a problem hiding this comment.
Providing meaningful alt text is important because it ensures images are accessible to people using screen readers and also helps when images fail to load, giving users essential context about the content or function of the image.
Can you give a more descriptive alt value to all images?
cjyuan
added
Reviewed
2 participants
Self checklist
Changelist
I have amended the following code inline with README file. There are three articles with links to seperate articles.