London |26-ITP-january | Dagim Daniel | Sprint 1 | wireframe#1174
London |26-ITP-january | Dagim Daniel | Sprint 1 | wireframe#1174Dagim-Daniel wants to merge 6 commits intoCodeYourFuture:mainfrom
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.
|
The Validation bot will complain about any changed files not in the Wireframe folder. Can you revert the changes made to the |
cjyuan
left a comment
There was a problem hiding this comment.
Can you take a look at this General Feedback to see if there
is anything you can do to make your PR more robust and ready?
Code is free of syntax errors! Well done.
Some areas that can be improved include
- Indentation of the code
- Typos
- Structure the page to better match the wireframe image. In particulars, align the image height between articles 2 and 3.
This reverts commit 85343a6.
You can execute the following commands to restore where Once you have successfully revert the changes made to |
cjyuan
left a comment
There was a problem hiding this comment.
Code is syntax error free. Page looks good. Well done.
Can you improve the indentation of the HTML code? Doing so can make the code easier to read.
You can try the VSCode's "Format Document" feature. It can help us format our code for better readability and consistency. To use the feature, right-click inside the code editor and select the option.

Learners, PR Template
Self checklist