Jun Wei• 200
@junwei-wong
Posted
Really clean solution. Some feedback on my end:
- In the design, I noticed that the heading font-size is bigger for larger screens and smaller for smaller screens. You can use the clamp function for the heading, e.g.
font-size: clamp(1.55rem, 5vw, 2.25rem);
. This should adjust to a bigger and smaller screen size. - The p tag in the header lacks a max width, something like
32rem
should do the trick. - For mobile screen size, the headers lacks a
text-align:center
.
Keep up the good work!
Marked as helpful
0