Completed using ReactJS

Community feedback

• 

  MikevPeeren•2,100

  @MikevPeerenPosted about 3 years ago

  Hey @kdavis010397 good job!

  Try to also add cursor pointer to your hover states and try to add transitions so they are less abrupt.

  Responsiveness is good, on a bigger screen however for example 2560px the page starts further in the middle but I think I like that.

  Also when you use images try to determine what kind of image it is before deciding if you should give an alt to it. The mobile app for example does not need an alt as for a screen reader this would be useless. More info here. https://www.w3.org/WAI/tutorials/images/

  Marked as helpful

  0

  ---

  

  P

  Kevin Davis•280

  @webdev1kevPosted about 3 years ago

  @MikevPeeren Thanks for the feedback!

  0
• 

  Yuko Horita•645

  @Sloth247Posted about 3 years ago

  This is just a question but why do you use form for the price card?

  0

  ---

  

  P

  Kevin Davis•280

  @webdev1kevPosted about 3 years ago

  I was thinking it was like submitting a request to get a download so I used form.

  0

  

  Yuko Horita•645

  @Sloth247Posted about 3 years ago

  @kdavis010397 This should not be form, form is normally used with input field inside it . Please see: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/form

  Also the button inside your form should not be button, has to be a link with <a> tag.

  Marked as helpful

  0

  

  P

  Kevin Davis•280

  @webdev1kevPosted about 3 years ago

  @Sloth247 Thanks!

  0