All comments

• 

  Luis Soto• 80

  @walkerlyna

  Submitted over 1 year ago

  x.x

  

  Challenge#5-Columns3

  2

  50

  

  P

  Piwko Marcin• 180

  @PiwkoO

  Posted over 1 year ago

  Hi your solution looks great, but I have a few suggestions:

  • For devices between 375px to 660px it would look better if you add an additional breakpoint
  • Your class naming would be better in English so when you join an international team you don't have to suddenly switch and you would be more used to the naming convention
  • Class names like [name]1, [name]2, etc are not very readable for other developers. It should be more descriptive so that when someone else is looking at your code it doesn't require to dive into html file
  • Your repeat the same code in different places. In that case, is better to create joined rules for a few classes and separate rules for the particular class.

  0