@vanzasetia
Posted
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 thep
element (page__pitch
) is very wide. So, I would recommend setting amax-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
@jaycgreenwald
Posted
@vanzasetia Thank you so much for the feedback!
@vanzasetia
Posted
@jaycgreenwald You're welcome! 👍