Responsive features card

Community feedback

• 

  P

  Kiara523•100

  @Kiara523Posted 3 days ago

  Your solution looks good and pretty close to the original one. The variable are very well organized and easy to read, also you ahve named the classes is pretty clear way. I would suggest to keep specificity on the same level as much as possible to avoid rules to be overridden by using more specific name for the element itself, an example the ".header h2 " it has an higher specificity, but giving it a class name would lower it, example header__title.

  Giving specifics width and height it can be dangerous as you did here.. '#container { width: 1272px; height: 735px; display: flex; flex-direction: column; justify-content: space-between; }' give a max-width and let the browser figure out the height can be a possibility, it helps with responsiveness.

  Great job over all. Well done!! I enjoying seeing how the same project can have different solution...

  0