Submitted over 2 years ago

Completed using ReactJS

Kevin Davis• 280

@webdev1kev

Preview site View code

This is a solution for...

Equalizer landing page

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Any feedback is welcome. Mainly working on methods to make sites responsive. Thanks!

Community feedback

MikevPeeren• 2,100

@MikevPeeren

Posted over 2 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

Kevin Davis• 280

@webdev1kev

Posted over 2 years ago

@MikevPeeren Thanks for the feedback!

Yuko Horita• 645

@Sloth247

Posted over 2 years ago

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

Kevin Davis• 280

@webdev1kev

Posted over 2 years ago

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

Yuko Horita• 645

@Sloth247

Posted over 2 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

Kevin Davis• 280

@webdev1kev

Posted over 2 years ago

@Sloth247 Thanks!

Please log in to post a comment

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord