-
-
Notifications
You must be signed in to change notification settings - Fork 412
London | January 2026 | Ihor Taradaiko | Sprint 1 | Wireframe #1037
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?
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.
👋 Hi @ihortar - thank you for submitting this PR for the wireframe exercise. The webpage correctly shows most of the exercise requirements. However, there are some code improvements to be made.
- Can you run your deployed page through W3 validator? This will give you a list of errors you can fix. If you aren't sure what an error means, you can search online or you AI (chatGPT, etc).
- Can you review the wireframe image from the onboarding exercise? Notice where the page title and description are aligned. Can you make your page look like this?
- The 'Read more' button for the wireframe article take me to a page that doesn't exist. Can you review this?
- The
alttext attribute for an<img>tag serves an important purpose. Can you ensure your images have alt text defined? Here is a useful article about alt text: https://www.w3schools.com/tags/att_img_alt.asp. - The lighthouse score for your webpage is 92. Can you see what needs to be done to make it 100?

Changelist