responsive landing

Community feedback

• 

  P

  Liam•360

  @asmthliPosted 4 months ago

  Hi Diego,

  Your solution looks great. Some things I think you could improve on though are the following:

  You have styled the "Perfume" heading by directly adding spaces and capitalising the letters in the HTML. There are properties in CSS to do this: 'letter-spacing' and 'text-transform'.

  I am not sure if it is a result of using React, but I think you could choose closer-matching HTML elements. For example the table row you have used to contain pricing information, it's not really tabular information and so a simple div with either flexbox or grid might have been better suited. The main argument for this approach would be accessibility, as a screen reader announcing a table is a little bit inaccurate.

  Hope this was somewhat useful.

  Marked as helpful

  0
• 

  P

  🔅 Yuliya 🐈•240

  @O-Julia-OPosted 4 months ago

  Perfect work!

  0