@vanzasetia
Posted
Hi there! 👋
Some suggestions from me.
- Add a
<title>
inside thehead
. The title of the page is the first thing that users will see before the site is fully loaded. Also, it is often the information that assistive technology will announce first. So, give your website a title. figure
element doesn't need to wrap everyimg
element. The only reason to usefigure
is if you need to include afigcaption
. Otherwiseimg
tag is fine.- The styling for the
.nav__cta.cta
element is complicated. So, I recommend simplifying it.- Set the element to
display: inline-block
to make it easier to style (because by defaulta
element is an inline element). - After that, you can use
padding
to give some width and height or simply make the button bigger. So, there's no need forwidth
andheight
. - The hover effect should exist on all screen sizes. I notice that on
807px
width the button doesn't have any hover effect. - Try to make the button looks closer to the original design. The background color is different from the design.
- Set the element to
- It looks like the styling of the interactive elements are complicated because of the
:hover
effect. If you can't make it simpler then I think it's okay. - Alternative text for images should not be hyphenated (not like code). Also, not all images need alternative text (e.g. illustrations).
That's it! I hope this helps!
Marked as helpful