Rohit Boseā¢ 30
@ROHIT-130SE
Submitted
It's nice to practice and learn more, I hope to work on more challenges at the earliest possible and please feel free to review and share your feedback on this work.
Karthik SP
@karthisp
All comments
- Karthik SPā¢ 110
@karthisp
Posted
Great work Rohit.
Just a few suggestions
- Give <html>, <body>, #container a height of 100% and align #container to the 'center' using align-items CSS property for flex.
Marked as helpful
0 - Kyleā¢ 50
@kylecloete
Submitted
This has been one of my favourite challenges to complete so far, making use of JavaScript and actually see you code do something was so interesting for me
When researching solutions for this challenge as well as looking at a couple of YouTube videos. I was left confused no offense to those people but what they were doing just seemed so convoluted for a beginner project.
I recommend looking at freeCodeCamps video "JavaScript Programming - Full Course"
Any Advice would be greatly appreciated especially regarding the structure of my html and appropriate tags to use for which section and how to clean up some of my JavaScript
Karthik SPā¢ 110@karthisp
Posted
Great Job.
Looks very close to the original design.
Just a few suggestions
- increase the 'letter-spacing' for button text
- Increase line height for <p>
- 1 rem margin for '.rating_component' and <h1> should be good enough
0 - Melvin Aguilar š§š»āš»ā¢ 61,480
@MelvinAguilar
Submitted
Hi there š, Iām Melvin and this is my solution for this challenge. š
š ļø Built With:
- TailwindCSS
- npm - prettier
Any suggestions on how I can improve and reduce unnecessary code are welcome!
Thank you. šāļø
Karthik SPā¢ 110@karthisp
Posted
Hi @MelvinAguilar you've done a great job with implementing the given design. Keep up the good work. I've got a lot of things to learn from you.
Just a small suggestion:
I see that you've used clip for sr-only (H1), instead try using clip-path. Apparently, that's what is recommended. https://developer.mozilla.org/en-US/docs/Web/CSS/clip-path
1
