@aszxqaz
Ami Rana
@amiranaAll comments
- @amirana
Clean code and almost close to the design. Great job, and keep it up! :)
- @notblezflzzz@amirana
Overall, it’s quite clean and well-structured. One small suggestion: instead of using a separate .card-header div, you could achieve the same effect with a border-top on the card itself.
Keep up the good work :)
- @Jabawaki90@amirana
You’ve created such a clean and well-organized layout, really nice work! One small suggestion: you could consider slightly reducing the font size for better balance. Other than that, everything looks great. Keep it up!
- @ndorokuda@amirana
Your design looks great overall!
Just a small suggestion: consider adjusting some of the spacing throughout the layout to improve the overall balance and readability.
Great work :)
- @carodg09@amirana
Great work! Your code is super clean and easy to read — really well done 👏
- @hussein430@amirana
Your work looks really clean and well-structured, well done!
- @fisherstack@amirana
Well done! Your work looks really clean and well-structured. Just a small note — the font size, color, and spacing are slightly off compared to the original design. Also, when using an <a> tag, it's best to always include an href attribute. If it's just a placeholder, you can use href="#" to keep the markup valid.
Keep up the great work!
Marked as helpful - @remainhumble@amirana
Hey, great job!
Just a small suggestion: you might want to remove the <br> tag from the HTML and use CSS instead to handle spacing and line breaks. This helps keep the HTML more semantic and separates content from styling.
Keep up the great work!
