Pricing Component with Toggle

Community feedback

• 

  Simon Davis•505

  @Skidragon

  Posted about 4 years ago

  Hi Ejovi, looks pretty good. :) I would suggest looking into em vs rem for CSS and the dangers of using fixed heights from Kevin Powell from Youtube. Also next time use a mobile first approach for your media queries, these things will save you more time on bigger projects. For the main content section, I would suggest a main element in place of a section element to communicate better that this is your main content.

  01
• 

  Mojtaba Mosavi•3,740

  @MojtabaMosavi

  Posted about 4 years ago

  @ruona88 good job, looks nice and reponds well. There are a few subtle things that you can improve such as.

  1. For "our pricing" use heading tag instead of <p>.
  2. Use a form tag for the switching functionality instead of a div since its more sementic.

  Happy coding :)

  02