All comments

• 

  José Alfonzo• 30

  @JoseAlfonzo92

  Submitted 8 months ago

  Feedback welcome.

  

  QR code component using HTML and CSS

  1

  4

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted 8 months ago

  Hey, The image isn't showing up. Try this code <img src="./images/image-qr-code.png" alt="Front-end development qr">

  Marked as helpful

  0

  This comment has received 2 repliesShow
• 

  MedHam94• 30

  @MedHam94

  Submitted 8 months ago

  hello, for me was too good, and the only issue I found was the <p> margin and padding, Even if I set the margin and padding to 0 in the first lines, the <p> still had them, Wichwas confusing for me,

  

  it is my first project on Frontend Mentor, i use Html and Css.

  1

  4

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted 8 months ago

  Hey, Good job on finishing this project.

  1. The margin on the <p> element is coming from the .paragraph class. You have set the margin-bottom:25px.
  2. There is another cancel order present at the top left corner of the screen.

  0
• 

  Harshit Srivastava• 10

  @studimeyt

  Submitted 8 months ago

  I am still quite unsure in display and positioning part and how they work exactly but will get around it eventually

  

  QR code using position and display

  1

  6

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted 8 months ago

  Hey, Good job on finishing this project. There is one thing which will improve your HTML code. Use semantic HTML For example instead of <div class= qrcode-box> use <main class="qrcode-box"> Same for <div class = "attribution"> use <footer class=attribution>

  Marked as helpful

  1
• 

  Nsoro Allan• 310

  @allancorp23

  Submitted 9 months ago

  So the question was good and easy but thanks for continuosly providing us with great questions that enhance our knowledge and skills.

  

  single-price-grid-component-master

  1

  6

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted 9 months ago

  Hey,

  1. Using h1 tag to point the main heading of the page is important to improve accessibility. so instead of <h4> Join our community </4>. use h1 tag <h1> Join our community </h1>

  0
• 

  Noyan Seçkin• 160

  @NoyanSeckin

  Submitted over 2 years ago

  

  Bootstrap

  2

  1

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted over 2 years ago

  Hey, You have uploaded the wrong live site URL. Please update the URL and generate the screenshot again. 😊

  0
• 

  Shmuel Amir• 150

  @ShmuelAmir

  Submitted over 2 years ago

  

  Article preview component

  1

  1

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted over 2 years ago

  Hey, You have uploaded the wrong live site URL. Please update the URL and generate the screenshot again. :)

  0
• 

  Adeniyi Smart• 520

  @Iamweird2

  Submitted almost 3 years ago

  hi, please review my code

  

  3column-preview-card-component flexbox

  2

  2

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted almost 3 years ago

  Hi, Maybe that's because of font-family.

  Try this

  body{ font-family: 'Lexend Deca', sans-serif; }

  Marked as helpful

  0
• 

  Faizal Nur• 35

  @faizalnur

  Submitted almost 3 years ago

  hi.. beginner front end here, i tried to post this challenge any feedback would be appreciated

  

  3-column preview card component

  1

  1

  0

  

  Anosh Sohail• 230

  @AnoshSohail

  Posted almost 3 years ago

  Well done :)

  0