Responsive landing page using Grid and Flexbox

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello there, Jason Greenwald! 👋

  Nice work on this challenge! 👍

  The aria-label on the social media links might not be appropriate. They are actually the social media links of the company. So, I would recommend making the aria-label as simple as possible. For example, for the Facebook link, aria-label="Facebook".

  Some more recommendations from me.

  • For the logo image and the hero image, I would recommend making them the actual image elements on the HTML instead of background images. I am sure that it will make it easier to style. Currently, you have to specify the size in order for the images to appear.
  • On 900px width, the text content of the p element (page__pitch) is very wide. So, I would recommend setting a max-width to prevent this from happening.
  • Write your code with a consistent style (e.g. the indentation, quotes, whitespace, etc). If you write your code that way, it will make it easier to read for everyone (including your future self). But, this can be done automatically by your code editor or an extension.

  That's it! Hope this helps. 😊

  Marked as helpful

  1

  ---

  

  Jason Greenwald•200

  @jaycgreenwaldPosted over 2 years ago

  @vanzasetia Thank you so much for the feedback!

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @jaycgreenwald You're welcome! 👍

  0
• 

  Jason Greenwald•200

  @jaycgreenwaldPosted over 2 years ago

  I read the accessibility report and added aria labels to the social media links. This was my first time using aria so feedback about that is appreciated.

  0