Anubhav Ray

Latest comments

• 

  Shaheed Wajee•220

  @wajeeNPCSubmitted about 2 years ago

  

  Interactive-rating-component

  1

  60
  

  Anubhav Ray•140

  @Anubhav-RayPosted about 2 years ago

  Hi @wajeeNPC

  Congrats on completing your first JavaScript project.

  I have some suggestions you might consider to improve your code.

  • To make your webpage accessible you should use atleast one main tag to cover your entire html and use forward slash to specify path for image and try to use semantic html.

  • Use alt tag for images that have some special meaning so that it might be helpful for screen readers.

  • Your rating is being submitted even if user has not selected any rating.Try to disable the submit button until user has selected the option.Its giving selected 0 out of 5.

  • Multiple ratings are being selected (or their color state is being changed) if one tries to click on every rating so try to fix it.

  • In submit button you have used spaces in html itself to make spacing between the letters of word submit.Try to use CSS letter-spacing property.

  • You should also write a detailed Readme so that others can gain additional insight on your project.

    Above all, The project is Well Done.I hope these tips will help you.
    

  Marked as helpful

  0
• 

  Justine Santos•10

  @hyxenonSubmitted over 2 years ago

  

  Media Query, Flex, Grid

  #react#tailwind-css

  1

  30
  

  Anubhav Ray•140

  @Anubhav-RayPosted over 2 years ago

  Hi @hyxenon. Congrats on Completing the Challenge.

  You can use picture tag for setting two different images.

  To make your webpage accessible you should use atleast one main tag to cover your entire html after body tag and should use semantic html

  Marked as helpful

  0
• 

  Lindelwe Khumalo•170

  @iSoyecodesSubmitted over 2 years ago

  

  Order-Summary-Component using css grid and flexbox

  1

  30
  

  Anubhav Ray•140

  @Anubhav-RayPosted over 2 years ago

  Hey @iSoyecodes for Responsive background pattern use Media Query.

  You can set default background-image of body tag as the image given for mobile background-imagein design if you make mobile-first design and use media query for setting body tag background-image as the desktop image for desktop design.

  Marked as helpful

  0
• 

  Juan Rodriguez•70

  @JuanRodriguezzSubmitted over 2 years ago

  

  Card component boostrap 5

  #bootstrap#accessibility

  1

  20
  

  Anubhav Ray•140

  @Anubhav-RayPosted over 2 years ago

  Hi @JuanRodriguezz Good Job Completing this Challenge.

  I have some suggestion you might consider to improve your code:

  1. You can make $149 bigger by using font-size property instead of using h5.

  2. You can add Cart Icon before the text Add to Cart by using Bootstrap Icons

  3. You should also try to make both prices horizontally aligned as given in design.

  Above all, The project is Well Done.I hope these tips will help you.

  Marked as helpful

  0