Frontend-Mentor-Four-card-feature-section

Community feedback

• 

  P

  Liam•360

  @asmthliPosted 4 months ago

  Hi,

  Nice work! I looked through your solution to try to find some things to improve on and came up with these few things (although they are a bit nit-picky):

  If you look at your Google Fonts import, you've actually listed a bunch of fonts to import (the ones from the previous challenges). I noticed I'd done this too. Google Fonts has a 'basket' of fonts which isn't cleared upon refreshing the page. I looked into this and modern browsers should avoid downloading imported fonts which haven't been used, so it's not really a problem but worth noting I suppose.

  I would have personally made a CSS selector the 'card boxes' in general. This would have saved you repeating a large amount of properties for each of the properties. It is really only the grid information which is specific to a given box.

  Hope this was of some help!

  Marked as helpful

  0

  ---

  

  Abigael74•40

  @Abigael74Posted 4 months ago

  Hi,

  Thanks for the advices.

  Have a nice day !

  0
• 

  akorir08y•110

  @akorir08yPosted 4 months ago

  Wonderful Design. Almost Similar to the design

  0