QR-component card with css-animation

Community feedback

• 

  Sounak Mukherjee•560

  @kanuos

  Posted over 2 years ago

  Hi Lesya, congrats on your first submission. The UI structure looks nice, however I've few questions/suggestions -

  1. Why is the card heading replaced by random lorem ipsum texts?
  2. Same goes with the card description
  3. Imagine a scenario - you visited a website where you are to scan a QR code (preferably by a phone camera). Would you like to hover the card before you can scan? There's no need for this effect. It is counter productive as it beats the purpose of a Quick Response card. The card rotation using perspective property is cool but kinda goes against the UX of this app.

  As far as the semantic HTML and BEM naming conventions are concerned - you aced it! At a quick glance, I thought it was perfect. Keep up the good work :)

  21

  Marked as helpful
• 

  Loai Esam•670

  @LoaiEsam37

  Posted over 2 years ago

  you have to put your <div class="qr__wrap"> tag inside a <main> tag so that it becomes easier for screen readers to know where is the main content and you can also use something like <nav> , <footer> and <sidebar> tags for the same reason.

  01

  Marked as helpful