Baki Jessy Justice Julien

Latest comments

• 

  Kate Dames•250

  @funficient

  Submitted over 1 year ago

  

  Order summary component with vanilla CSS

  2

  60
  

  Baki Jessy Justice Julien•110

  @blackdreamer15

  Posted over 1 year ago

  Hey, @funficient! Great work out there.

  You do not actually need to create that illustration. It is part of the assets. In case you do not have those, The link below refers you to my GitHub where you can get those assets: pattern-background-desktop.svg and pattern-background-mobile.svg.

  Click on this link.

  Best of luck! 😊

  0

  Marked as helpful
• 

  Yash Bajaj•230

  @YaSh8202

  Submitted about 3 years ago

  

  Simple QR Code

  1

  10
  

  Baki Jessy Justice Julien•110

  @blackdreamer15

  Posted about 3 years ago

  Hi @YaSh8202! Congratulations on completing this challenge. I checked your code on GitHub and have some suggestions to make.

  1. You need to add the alt attribute to the img tag to solve the HTML issue.

  2. You have to use a <h3> tag instead of a <h3> tag to resolve one of the accessibility issues. Conventionally, you have to start with a level-one heading. (that is, the h1 tag)

  3. For the semantic reasons, it will be best to change <div class="center"></div> to <main class=""></main> since it contains the main content of the page.

  4. I think the <div class="card"></div> is not really important since you can apply the styles directly to the <main class=""></div>.

  5. For the <div class="attribution></div>, it will be best to use <footer class="attribution></footer> instead to resolve one of the accessibility issues.

  Good luck with your other challenges.

  0

  Marked as helpful
• 

  Kermit Costa•20

  @kermitcosta

  Submitted about 3 years ago

  

  QR code component using HTML5 & CSS3

  #styled-components

  1

  30
  

  Baki Jessy Justice Julien•110

  @blackdreamer15

  Posted about 3 years ago

  Hi @kermitcosts! Congratulations on completing this challenge. I checked your code on GitHub and have some suggestions to make.

  1. You have to use a <h1> tag instead of a <p class="first-text"></p> to resolve one of the accessibility issues.

  2. For the semantic reasons, it will be best to change <div class="main"></div> to <main class="main" role="main"></main> since it contains the main content of the page. Let it embed the whole main content of the page.

  Good luck with your other challenges.

  0

  Marked as helpful
• 

  Barbie_Dev•30

  @urrrch

  Submitted about 3 years ago

  

  qr-code-component-main

  1

  20
  

  Baki Jessy Justice Julien•110

  @blackdreamer15

  Posted about 3 years ago

  Hi @urrrch! Congratulations on completing this challenge. I checked your code on GitHub and have some suggestions to make.

  1. You have to use a <h1> tag instead of a <h4> tag to resolve one of the accessibility issues. Conventionally, you have to start with a level-one heading. (that is, the h1 tag)

  2. For the semantic reasons, it will be best to change <div class="content"></div> to <main class="content"></main> since it contains the main content of the page.

  3. I think the <div class="wrapper"></div> is not really important since you can apply the styles directly to the <main class="content"></div>.

  4. Also, you can increase the dimensions of the container and its items to make the whole QR code component a bit bigger.

  5. You can also add some padding to the content to add some spaces between them.

  Good luck with your other challenges.

  0