@R3ygoski
Posted
Hello Roni, congratulations on completing your project, it turned out really well.
I noticed that in your CSS there are some redundancies, such as the use of certain position
properties that weren't having any effect on either mobile or desktop.
Regarding your HTML, I noticed that it has good semantics, but there are some sections without semantic meaning, for example these two: <div class="preparation">
and <div class="table__container">
, both could be <section>
instead of <div>
.
If you want to make your CSS cleaner, I recommend using @import, here's a link about it: W3School - Import, this way you can divide it into sections, including separating variables from the main part.
I also recommend CSS Nesting, as it helps with organization, here's another link about it: MDN - CSS Nesting
And congratulations on your project, it's really good, keep practicing and improving, if you have any questions, just ask below.