@CodeWithAlamin
Posted
Hi GGS👋 Great job on completing this challenge! 🥳
I would like to share a few suggestions on this solution if you don't mind.
-
Use semantic HTML tags: Some of the HTML tags used in the code, such as
div
, do not convey any meaning about their content. Using semantic tags, such asheader
,main
,section
, andfooter
, can help make the code more readable and improve the accessibility of the page. -
Use BEM naming convention: The code uses class names that are not very descriptive, such as
left
,center
, andright
. Using the BEM (Block-Element-Modifier) naming convention can help make the class names more meaningful and easier to understand. For example,review__stars
instead ofstars
andreview__by-review
instead ofby-review
. -
Avoid using inline styles: Inline styles, which are applied directly to an element using the
style
attribute, can make the code harder to maintain and override. It's generally better to define styles in a separate stylesheet and apply them using class names or other selectors.
Overall, this is a very well done solution to the challenge. Great job!
I hope this feedback was helpful. 😊 Keep up the good work!👍
Marked as helpful
@Guilherme-Goncalves-de-Souza
Posted
Thank you very much!
Gratitude for all the feedbacks, I will take care of them in my next projects.
Thank you very much @CodePapa360 !!!