All comments

• 

  Diogo Pegorel•10

  @DiogopegorelSubmitted over 1 year ago

  

  qr code challenger

  2

  60
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  salam o Alaikum! @Diogopegorel. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap the code

  <div class="container">

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element. Replace this

  <p class="p1">Improve your front-end <br>skills by building projects</p>

  ▪ Instead of using the <br> tag , you should use a semantic HTML element and the CSS margin or padding properties.

  CSS: 🎨

  📍▪ Check the style guide & change the background color.

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  kirubavathi-VM•10

  @kirubavathi-VMSubmitted over 1 year ago

  

  QR-code-component

  3

  40
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @kirubavathi-VM. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap the code

  <div class="container">

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element. Replace this

  <h5>Improve your font-end <br>skills by building projects</h5>

  ▪ Instead of using the <br> tag , you should use a semantic HTML element and the CSS margin or padding properties.

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  📍▪ Check the style guide & change the background color.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  0
• 

  Lucas Paulino Marra•10

  @lucaspmarraSubmitted over 1 year ago

  font-weight: 400 changed nothing, something to worry about?

  

  QR Code using HTML + PostCSS

  3

  80
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @lucaspmarra. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It's necessary for images to have short, descriptive alt text so screen reader users clearly understand the image's contents and purpose.

  <img class="card-image" src="./images/image-qr-code.png" alt="">

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  1

  This comment has received 1 replyShow
• 

  atiqurrehman123•10

  @atiqurrehman123Submitted over 1 year ago

  

  qr-code-component-main

  1

  30
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @atiqurrehman123. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap the code

  <div class="container">

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element. Replace this

  <h5 class="card-heading">Improve your front-end <br />skills by building projects </h5>

  ▪ Instead of using the <br> tag , you should use a semantic HTML element and the CSS margin or padding properties.

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  0
• 

  Rishabh Jain•20

  @Rishabh2324Submitted over 1 year ago

  

  QR Card using pure HTML and Css 🤓

  1

  30
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @Rishabh2324. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element.

  Replace this

  <h6 class="qr-details-title"> Improve your front-end skills by building projects </h6>

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  0
• 

  Abdelrahman Mohamed sayed•10

  @Abdelrahman-Muhammed13Submitted over 1 year ago

  

  First mentor QR Code Component

  2

  20
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @Abdelrahman-Muhammed13. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap the code

  <div class="main-container">

  ▪ It's necessary for images to have short, descriptive alt text so screen reader users clearly understand the image's contents and purpose.

  <img src="../image-qr-code.png" >

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  0
• 

  Gustavo Atanazio•80

  @gustavo-atanazioSubmitted over 1 year ago

  

  QR Code - first FEM challenge

  2

  20
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @gustavo-atanazio. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag.

  Click & Wrap the code

  <div class="container">

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element.

  Click & Replace

  <span class="title">Improve your front-end skills by building projects</span>

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  0
• 

  Marko•10

  @terza85Submitted over 1 year ago

  

  Responsive web page using Flex box with Sass

  1

  30
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @terza85. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap the code

  <div class="main-container">

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element. Replace this

  <p class="heading">Improve your front-end skills by building projects</p>

  ▪ It's necessary for images to have short, descriptive alt text so screen reader users clearly understand the image's contents and purpose.

  <img src="images\image-qr-code.png" alt="" class="src">

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  0
• 

  peredopr•20

  @peredoprSubmitted over 1 year ago

  

  This is my solution to this fun qr-code challenge!

  2

  40
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  @peredopr. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It's necessary for images to have short, descriptive alt text so screen reader users clearly understand the image's contents and purpose. Click and Change

  <img src="./qr-code-component-main/images/image-qr-code.png">

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap

  <div class="div-master">

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element. Click & Replace

  <h2>Improve your frond-end skills by building your projects</h2>

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  0
• 

  Hristo Petrov•120

  @King0fBladesSubmitted over 1 year ago

  

  QR code component

  2

  30
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @King0fBlades. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  1
• 

  Ignat Marius•300

  @Mariusq17Submitted over 1 year ago

  

  qr code design

  2

  40
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @Mariusq17. 🎯Congratulations on completing the challenge !

  💬 I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap

  <div class="main-container center">

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Victor616•20

  @VictorR616Submitted over 1 year ago

  

  Simple css card

  1

  40
  

  RabicaTahir•500

  @RabicaTahirPosted over 1 year ago

  Asalam o Alaikum! @VictorR616. 🎯Congratulations on completing the challenge !

  💬I have some suggestions about your code.

  HTML 🔔:

  ▪ It is advisable that you place all your main site content in a landmark element <main>. Wrap the body code in main tag. Click & Wrap

  ▪ Best practice is to ensure that the beginning of a page's main content starts with a h1 element. Replace this

  CSS: 🎨

  📍▪ Instead of using pixels in font-size, using relative units like em or rem is a good practice.

  I hope you find it useful! By the way, the solution was great. 🙌💯

  Enjoy coding, Stay Safe! 🤞

  Marked as helpful

  1