@TheManDemiSubmitted 4 months ago
Petrakowww
@PetrakoowAll comments
- @PetrakoowPosted about 2 months ago
Hi, I've just looked at your work, well done, but I see obvious flaws in the choice of colors. For example, the background or text inside the cards. They should be changed, the color of the text of the cards is taken from the main header text and transparency is applied to it. It is written in the readme about styles. When the screen narrows, the icons also begin to shrink at a certain point, I assume that because of the display flex without the shrink used.
In general, the work is good, you just need to adjust the choice of colors in the styles.
0 - @AsadullahBadhanSubmitted 8 months ago@PetrakoowPosted about 2 months ago
I really liked the implementation of the site adaptation depending on the screen zoom change. Initially, I did not know how to do this correctly using Flexbox, and the result did not come out as I had planned. Now I understand that it's worth studying Grid and rewriting some of the layout and styles. Thank you for your example — it has become a great reference point.
I also noticed that you used Gulp and other tools to work, it's impressive! I wish you success in your future projects!
0 - @RubsnewgateSubmitted 9 months ago
- @HamptooonSubmitted 2 months ago@PetrakoowPosted 2 months ago
Very good work, the layout is accurately rendered. Neat and beautiful!
Marked as helpful0 - @ahmedEid6Submitted 2 months ago
- @amyrhexaSubmitted 10 months ago@PetrakoowPosted 2 months ago
Hi, good work! However, there are some small comments. The first thing that catches my eye is the weight of the links, it is less than on the layout. There are also inaccuracies in the size of the location block, as well as the indents between the main block transitions. Good luck!
1 - @Lazur05Submitted 2 months ago
- @mohammed-b-hassanSubmitted 12 months ago@PetrakoowPosted 2 months ago
Well done, but the table of contents of the card is not bold and the background is not full page
2
