Justin Gorelik

Latest comments

• 

  Dmitriy Smirnov•40

  @Teor99

  Submitted about 3 years ago

  

  VSCode + LiveServer + Chrome(PerfectPixel), raw html and css (Newbie)

  2

  40
  

  Justin Gorelik•100

  @jgorelik23

  Posted about 3 years ago

  Привет братан!

  One suggestion that I have for you is to center the QR code component not only vertically, but horizontally as well. This can be achieved by adding the following code to your CSS:

  html, body {
      height: 100%;
  }
  
  body {
      justify-content: center; /* add this in addition to existing styles for the body */
  }
  

  Read more about flexbox: https://css-tricks.com/snippets/css/a-guide-to-flexbox/

  Great work!

  1

  Marked as helpful
• 

  Marvel Victor•390

  @Mharvel13

  Submitted about 3 years ago

  

  NFT Preview Card With Flexbox

  1

  50
  

  Justin Gorelik•100

  @jgorelik23

  Posted about 3 years ago

  Hey Marvel!

  Here's some feedback:

  • Something I would recommend looking into would be the CSS min() function, which (in some cases) can be used to replace media queries.
  • Try replacing some of your divs with semantic HTML elements.
  • Check out the accessibility issues shown in the report from Frontend Mentor.

  Great work!

  Learn more: https://developer.mozilla.org/en-US/docs/Web/CSS/min

  0

  Marked as helpful