Responsive single price page with Sass

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Caique Carvalho da Silva! 👋

  Good job on this one! 🎉 Your solution responds well and overall looks good. Here's my suggestions:

  • The div element has no special meaning at all. Use of more appropriate elements instead of the div element leads to better accessibility for readers and easier maintainability for authors. For .header-title use heading <h1> - <h6> element. For .header-subtitle you can use paragraph <p> element. This also applies to text in other sections.
  • On mobile resolution add width: 100% to .main-container and remove max-width: 400px so your container will stretch across different screen sizes. Keep max-width: 660px on 660px media query to prevent it for stretching too much.
  • Use unordered list <ul> for .reasons-list. HTML lists are used to present list of information in well formed and semantic way.

  Good luck with that, have fun coding! 💪

  Marked as helpful

  1
• 

  Ahmed Faisal•5,095

  @afrusselPosted over 3 years ago

  Use color picker add on to pick color from images file

  Marked as helpful

  1