@out0fSpac3
Hi! I’m a frontend developer with 2+ years of React experience. I’m also a Programming Coach at Coding Giants, teaching kids about web development and AI. I’m always eager to offer feedback and support to fellow coders on their journey.
I’m currently learning...I’m currently diving into AI backend development to broaden my skill set and explore new tech frontiers. Additionally, I’m participating in a Three.js 3D programming bootcamp, where I’m excited to create immersive, interactive web experiences that push the boundaries of frontend technology.
Latest comments
- @Radexman
Hey! Very nice job on that styling, it looks very similar to the design the only difference that I can spot is the bold font on button. I also looked through you code, there is no need to give alt descriptions for icons that don't give important context for overall accesibillity and the h4 headings should not be used without heading hierarchy.
Here is some further reading on the topic: https://www.contentkingapp.com/academy/headings/
Overall amazing job on getting your design so close to the original. Good luck on next projects and happy coding!
Marked as helpful - @sithum-gimhara@Radexman
Hi there Sithum!
Good job on that project! If you don’t mind, I will provide you with some feedback :)
So I checked your code and it appears that you used <h3> before <h1> tag and <h1> is repeated twice. This is considered a mistake because when using headings we want to stick to the so - called hierarchy. You probably did it because the font size was correct but you can just change it in css in no time. We typically want to start from h1 because it should be a main title on our page and move our way to h6. Here is some good resource on that topic:
https://www.contentkingapp.com/academy/headings/
Aside from that the code looks good, you have a good naming convention for classes. Very readable and self-explanatory.
Overall amazing work on that project! I hope I was a little bit helpful for you. Good luck on next projects and happy coding!
Marked as helpful - @azizp128@Radexman
Hi there mate! Good job on that project! If you don’t mind, I will provide you with some feedback :)
So I checked your code and it appears that you used <h3> and skipped the <h1> tag. This is considered a mistake because when using headings we want to stick to the so - called hierarchy. You probably did it because the font size was correct but you can just change it in css in no time. We typically want to start from h1 because it should be a main title on our page and move our way to h6. Here is some good resource on that topic:
https://www.contentkingapp.com/academy/headings/
Aside from that the code looks good, you have a good naming convention for classes. Very readable and self-explanatory.
Overall amazing work on that project! I hope I was a little bit helpful for you. Good luck on next projects and happy coding!
Marked as helpful - @AbdoX9@Radexman
Hi there, good job on that project! If you don’t mind, I will provide you with some feedback. So I checked your code and it appears that you used <h2> tag before <h1>. This is considered a mistake because when using headings we want to stick to the so - called hierarchy. You probably did it because the font size was correct but you can just change it in css in no time. We typically want to start from h1 because it should be a main title on our page and move our way to h6. Here is some good resource on that topic:
https://www.contentkingapp.com/academy/headings/
Aside from that the code looks good, you have good naming convention for classes, there is indentation and some good whitespace going on. Very readable and self-explanatory.
Overall great work on that project! I hope I was a little bit helpful for you. Keep going and happy coding!
- @Rioba-Ian@Radexman
Hi there, good job on that project! If you don’t mind, I will provide you with some feedback. So I checked your code and it appears that you used <h3> tag before <h1> and <h1> is repeated twice. This is considered a mistake because when using headings we want to stick to the so - called hierarchy. You probably did it because the font size was correct but you can just change it in css in no time. We typically want to start from h1 and move our way to h6. Here is some good resource on that topic:
https://www.contentkingapp.com/academy/headings/
Aside from that the code looks good, you have good naming convention for classes, there is indentation and some good whitespace going on. Very readable and self-explanatory.
Overall great work on that project! I hope I was a little bit helpful for you. Keep going and happy coding!
- @Odo-Peter@Radexman
Hi there, good job on that project! If you don’t mind, I will provide you with some feedback. So I checked your code and it appears that you used <h3> tag and no <h1>, this is considered a mistake because when using headings we want to stick to the so - called hierarchy. You probably did it because the font size was correct. No worries just add <h1> element and change its styling in css. We typically want to start from h1 and move our way to h6. Here is some good resource on that topic:
https://www.contentkingapp.com/academy/headings/
Also you just need to change the font sizes and font weights and your project will be as close to original as possible! But that's a really minor thing.
Overall great work on that project! I hope I was a little bit helpful for you. Keep going and happy coding!
Marked as helpful
