Piotr Pieprzyk• 10
@PiotrPieprzyk
Posted
Overall, it looks good, there are small things that I would improve.
- The border-radius for the image is a bit too large. What's more you don't have to repeat 20px 4 times, you can enter 20px once.
- You could try to install an extension such as eslint or prettier to automatically improve the style of your code. I saw unnecessary nesting in the css file or an empty style under the .main { class
- if you would like to use the BEM methodology, I think the .text-block__large class should be called .text-block--large. --large is more of a modifier.
Have a nice day!
0
VovaKuzmenko• 40
@VovaKuzmenko
Posted
Thank you, I'll definitely take this into account.
0