@martin-tercero1Submitted 6 months ago
yyuntzpan
@yyuntzpanAll comments
- @yyuntzpanPosted 2 months ago
Well done!
There are some areas in the design where the font settings could be fine-tuned.
The font sizes and weights deviate slightly from the design. I suggest reducing the font size of the titles and increasing the font weight to create a stronger visual hierarchy.
The spacing between sections is inconsistent and appears to be too narrow. Consider using padding, margin, or flexbox with gap to adjust the spacing and align it more closely with the design. This will create a more visually appealing and user-friendly layout.
0 - @fivetailsdevelopmentSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I'm happy with this one being a close reproduction of the original, with some additional tweaks such as using a hover transition rather than an immediate colour switch. Next time I'm going to think a bit more in advance on what custom variables and classes I want to set up in advance to make it easier to keep the code consistent.
What challenges did you encounter, and how did you overcome them?I think there's a better solution for putting the right gaps between the elements other than putting margins under each element (without having to use grid or flex for such a simple design), though it did give me my first use case for the all-children selector, which was a good learning.
What specific areas of your project would you like help with?Nothing specific this time.
@yyuntzpanPosted 3 months agoYour layout matches the design very well, that's impressive!
Next time, you could try the challenge of adapting font sizes for different devices without using @media queries.
0 - @rkumarkraviSubmitted 7 months ago@yyuntzpanPosted 7 months ago
You did a great job. If I were to give a suggestion, you might want to consider the font size of the title. A font that is too large can result in automatic line breaks. In terms of design, it might not be ideal for this short paragraph to be split into three lines. You could try adjusting it and see how it looks!
0
