-
-
Notifications
You must be signed in to change notification settings - Fork 264
CapeTown | May-2025 | Wahae Koela | Wireframe #710
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
CapeTown | May-2025 | Wahae Koela | Wireframe #710
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- When a wireframe is provided, our implementation should closely reflect its appearance and layout to ensure consistency with design expectations. You're off to a solid start. To better align with the wireframe, can you layout the articles so that
- The 1st article is on top of the 2nd and 3rd articles
- The left edge of the 1st article is aligned with the left edge of the 2nd article
- The right edge of the 1st article is aligned with the right edge of the 3rd article
-
According to https://validator.w3.org/, there is a warning in your code. Can you fix it?
-
When the page is fully scrolled to the bottom, the "Read more" link of the bottom article is partially blocked by the footer. Can you address this issue?
-
To follow best practices, in your PR description, can you provide a brief description (under the "Changelist" section) summarizing the purpose of the PR and the changes you’ve made?
Changes look good. Well done! Please note that in CYF courses, the recommended way to inform the reviewer of your changes is to do both of the following:
Also, can you find out from ChatGPT the following
and practice what you find in future PRs? |
Learners, PR Template
Self checklist
Changelist
extra section tag removed
made sure correct article order
fixed the W3C issue
prevented the footer overlapping
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.