Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request failed with status code 502
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Pricing Component with Toggle Master

Tony 140

@handleryouth

Desktop design screenshot for the Pricing component with toggle coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


open to any advice, thank you!

Community feedback

@AgataLiberska

Posted

Hi @handleryouth! Your solution looks good, well done! Only on smallest mobile sizes the content of the cards overflows the cards themselves, and the toggle is really small, would be good if it got resized for smaller screens maybe. And the middle card still has a bigger padding on mobile, not sure if that's a part of the design or not.

Make sure that you don't leave the inputs without a <label> or an aria-label. It could also use some focus styles, the outline shouldn't really be removed without providing an alternative.

I would also change the <h1> elements where you have the different prices to a different element, not a heading. You should only have one <h1> per page, and the rest of headings should make sense in the hierarchy of the page.

Hope this helps, let me know if anything needs more explanation and I'll do my best to help :)

Marked as helpful

0
Tony 140

@handleryouth

Posted

Hi there! Thank your for the feedback mate. You are right, the website is kind of not resized well in the mobile device. Before posting the solution, i asked my friends whether the website showed well in their devices. And in found it's okay. But, your feedback showed that it is not, so i need to think back about the reponsive design. I am still learning about how to use the media inquery properly, it always one of my biggest concerns in every website. About the padding in the middle price tag, i think i forgot to remove the padding. Anyway, thanks a lot mate!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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