@CodePapa360
Posted
Hi Péter Balázs👋 Great job on completing this challenge! 🥳
Overall, the code looks good and the webpage appears to be functioning as intended. However, there are a few things that can be improved to make the code more efficient and maintainable:
-
Use semantic HTML elements: Instead of using
div
elements for everything, consider using more semantic HTML elements such asheader
,main
,section
,article
,aside
, andfooter
to better convey the meaning and purpose of the different parts of the webpage. For example, thestats
element could be anaside
element, and thecard
element could be asection
. -
Use BEM (Block Element Modifier) methodology for class names: BEM is a naming convention for CSS class names that helps to prevent naming conflicts and makes the code more readable and maintainable. Instead of using multiple classes with hyphens (e.g.
stats | text-center align-center-sm-only
), consider using BEM syntax where the block is separated from the element by a double underscore (__
) and the modifier is separated from the block or element by a single underscore (_
).
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
@PBalazs0517
Posted
@CodePapa360 Thank you for the feed back!
@CodePapa360
Posted
@PBalazs0517 Your are welcome!!