air091• 140
@air091
Submitted
@tanawutw03
@air091
Submitted
@tanawutw03
Posted
The project looks great. Here are a couple of my feedback:
CSS Naming Convention: I see you're using class names like text-preset-1
and text-preset-2
. While it's functional, using more descriptive names (like .main-heading
or .subtext
) could make the code more readable and easier to maintain, especially as the project grows.
Unused CSS: I noticed that .text-preset-2
is defined in the stylesheet but isn't applied to any element in the HTML.