CSS grid, Flex-box. positioning and pseudo class

Community feedback

• 

  Doston Nabotov•950

  @dostonnabotovPosted about 2 years ago

  Hi, there! that looks great!

  Little advice: minimum width for mobile is 320px. The most common one is 375px. You don't have to worry about sizes lower than that unless you're designing a site for the watch.

  Don't forget to remove the default margin on the body. It is creating small overflows.

  Also, never use more than one @media query for one element unless you really need to. You have created 3 @media queries for .storage-container, which is over the limit.

  Last, but not least, try to stick with common naming conventions, such as BEM. Because other developers, who analyze your code can easily understand your code without difficulty.

  I hope it helps. Good luck!

  Marked as helpful

  0