All comments

• 

  Derrick Yew Choon Min•20

  @yewcm123Submitted over 2 years ago

  

  Interactive Rating Component

  1

  20
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @yewcm123, Congratulations on completing this challenge! I really liked the result of your project, but I have some suggestion that I like to share: -

  1> <div><span>2</span></div> Selected rating no is not visible. Change text color during selection.

  2> Add alt description or alt="" aria-hidden="true".

  You can go through my solution if you like. Solution Link

  ✌️ I hope this helps you. Happy Coding

  1
• 

  samlors•60

  @samlorsSubmitted over 2 years ago

  

  Interactive raiting component

  2

  20
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @samlors, Congratulations on completing this challenge! I really liked the result of your project, but I have some suggestion that I like to share: -

  1> Increase h1 font-size & decrease <p> line height.

  2> Add <p> margin-top or h1 margin-bottom as both are too close with each other.

  You can go through my solution if you like. Solution Link

  ✌️ I hope this helps you. Happy Coding.

  0
• 

  Maxwell Takyi•250

  @damagermaxSubmitted over 2 years ago

  

  Responsive News Homepage - Vanilla CSS

  1

  50
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello Maxwell Takyi, Congratulations on completing this challenge! I really liked the result of your project, but i have some suggestion that i like to share:-

  1> Increase <p> font-size in Max-width: 535px .

  2> Nav should be hidden in mobile resolution but it still display in right side.

  You can go through my solution if you like. Solution Link

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  0
• 

  Sahiru•220

  @bhatsahil13579Submitted over 2 years ago

  

  rating page

  1

  40
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello Sahiru, Congratulations on completing this challenge!

  I really liked the result of your project, specially it returns back to start page after showing rating.

  My suggestion - add little more padding to main-container on bottom .

  ✌️ I hope this helps you. Happy Coding.

  1
• 

  Mennatallah Hesham•1,470

  @Mennatallah-HishamSubmitted over 2 years ago

  

  Responsive advice generator

  1

  40
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @Mennatallah-Hisham, Congratulations on completing this challenge!

  I really liked the result of your project, but it seems that on clicking the cube, quote doesn't change.

  You can go through my solution if you like. [Solution] (https://www.frontendmentor.io/solutions/advice-generator-app-using-bootstrap-and-js-SVBqjXVSYG)

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  0
• 

  Abd Almalik jakmery•280

  @abdalmalik1996Submitted over 2 years ago

  

  Product preview card component

  1

  90
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Great job. Keep it up

  0
• 

  Adam Wilder•110

  @wilderav86Submitted over 2 years ago

  

  QR code component using HTML, CSS, and Sass

  #sass/scss

  1

  60
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello Adam Wilder, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  1
• 

  Yuri de Oliveira•140

  @yurideoliveira2712Submitted over 2 years ago

  

  HTML e CSS Responsive site

  2

  60
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @yurideoliveira2712, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  Happy Coading

  Marked as helpful

  0
• 

  dedsec-ninja420•50

  @dedsec-ninja420Submitted over 2 years ago

  

  Project made by using HTML, SASS, CSS Flexbox, CSS Grid and Bootstrap

  #sass/scss#bootstrap

  1

  30
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @dedsec-ninja420, Congratulations on completing this challenge!

  I really liked the result of your project, specially you implemented Bootstrap as well.

  Happy Coading

  0
• 

  Anubhav Ray•140

  @Anubhav-RaySubmitted over 2 years ago

  

  Product Preview Card

  #bootstrap

  3

  50
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello Anubhav Ray, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  Q. How do I make this website according to mobile design?

  =>Use the media query property to layout according to mobile design and modify the max width that you want the styles to take effect. It will allow you to modify component's property accordingly.

  example-

  @media only screen and (max-width: 600px) {
    body {
      background-color: lightblue;
    }
  }
  

  Q. How do I pull up the strike through price so it is inline with the bold price as given in the design for desktop

  =>

  <div class="textalign">
      <span class="span1">$149.99 </span>
    <span class="span2"><s>$169.99</s></span>
    </div>
  

  css-

  .textalign {
      text-align: left;
      display: flex;
      align-items: center;
  }
  

  You can go through my solution if you like click here

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  1
• 

  Dede Maulana•20

  @dhemolSubmitted over 2 years ago

  

  Responsive QR Code using CSS Flex

  #accessibility#laravel#next#typescript#mongodb

  2

  50
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  HelloDhemol, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  3- A meta element with an http-equiv attribute whose value is X-UA-Compatible must have a content attribute with the value IE=edge. Context:

  UTF-8> <meta http-equiv=X-UA-Compatible content=IE=7> `

  You can go through my solution if you like click here

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  1
• 

  mehmet Alptekin Isiklar•80

  @aisiklarSubmitted over 2 years ago

  

  responsive web site with vanilla JS

  2

  30
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello mehmet Alptekin Isiklar, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  3- Bad value `` for attribute src on element script: Must be non-empty. Context: </div> </scri

  4- Consider adding a lang attribute to the html start tag to declare the language of this document. Context:

  TYPE html><html><head>

  You can go through my solution if you like click here

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  1
• 

  Micheal Arumgbo•40

  @michealarumgboSubmitted over 2 years ago

  

  Responsive Ratings Card Using CSS

  1

  30
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello Micheal Arumgbo, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  You can go through my solution if you like click here

  ✌️ I hope this helps you. Happy Coding.

  0
• 

  Musharaf Aahel A•20

  @musharafaahelSubmitted over 2 years ago

  

  Using Intermidiate HTML and CSS

  2

  30
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @musharafaahel, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  Also its not mobile responsive. You can go through my solution if you like click here

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  0
• 

  deleete1•20

  @Deelite34Submitted over 2 years ago

  

  QR code page using flex box

  3

  50
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @Deelite34, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  Also answer to your question - By default font-weight for headings in browsers is “bold” (or more specifically, 700). Thats why it will make no change.

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  1
• 

  Exileurt•100

  @ExileurtSubmitted over 2 years ago

  

  CSS and HTLM

  3

  50
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @Exileurt, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- Images must have alternate text

  Context: <img src="images/icon-cart.svg" id="cart"> click here

  Also its not mobile responsive. You can go through my solution if you like click here

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  1
• 

  Betty H•40

  @bh-codesSubmitted over 2 years ago

  

  Product Preview Card

  3

  40
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello Betty H, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  As you already make the corner rounded, so no need to repeat.

  ✌️ I hope this helps you. Happy Coding.

  Marked as helpful

  0
• 

  Giorgi Bebia•50

  @Giorgi-BebiaSubmitted over 2 years ago

  

  NFT Preview Card Component

  2

  30
  

  Amulya Prakash•800

  @amulyalovescodingPosted over 2 years ago

  Hello @Giorgi-Bebia, Congratulations on completing this challenge! I really liked the result of your project, but I have some tips that I like to share:

  1- Every page should have one main landmark <main>. So replace the div that wraps the whole content with <main> to improve the accessibility. click here

  2- All page content should be contained by landmarks, you can understand better by clicking here: click here

  ✌️ I hope this helps you.

  Marked as helpful

  0