Finish KATVR landing page - #184
Conversation
DanielCamposDevX
left a comment
DanielCamposDevX
left a comment
There was a problem hiding this comment.
Demo link was not provided correctly, remember to check before submitting to review
|
Hello! Thank you for your feedback. I fixed the DEMO LINK and verified that it is working correctly now. I also redeployed the project to GitHub Pages. Could you please review it again? Thank you for your time! DEMO LINK |
DanielCamposDevX
left a comment
DanielCamposDevX
left a comment
There was a problem hiding this comment.
Landing page seems fine just remember to put the real logo of KAATVR provided on figma and ajust the fonts of the page to match figma.
The real problem is the page is missing the buy phase, where you should click Buy Now and go to the buy phase (check figma laayout)
|
Hi! I’ve made the requested updates:
The updated demo is working now. Could you please review it again? Thank you! |
raulriato
left a comment
raulriato
left a comment
There was a problem hiding this comment.
The only thing you still didn't add is the favicon.
I'll approve it, since is no major problema, but recommend you add it.
| name="viewport" | ||
| content="width=device-width, initial-scale=1.0" | ||
| /> | ||
| <title>Title</title> | ||
| <title>KATVR</title> | ||
| <link | ||
| rel="stylesheet" | ||
| href="src/styles/main.scss" | ||
| /> | ||
| </head> |
There was a problem hiding this comment.
The favicon is missing. Choose any image you already have on the project, it doesn't need to be any specific one, as long as it looks fine on the tab.
3 participants
DEMO LINK.
Hello!
I have completed the project and reviewed it carefully. I also ran the tests, and they passed successfully.
I did my best to match the Figma design and make the layout responsive.
If there are any improvements or corrections needed, I would really appreciate your feedback so I can continue improving my front-end skills.
Thank you for your time and review!