Easybank landing page using HTML, CSS and JS

I'd also check out the images on desktop as they all appear to get squished and not stay in the original ratio. This is for all images on the desktop version of the site.

Hey, great work on this one. The layout in desktop is good, resizes well and the mobile state is good as well.

Some suggestions would be that:

1. Your navbar links should be wrapped inside their own a tags and not just in li tags. Since they are links right, they should be nested with it.

2. Your buttons should have a :focus state because you have removed the outline property. This outline helps user to see where they are at in your website especially when using keyboard. If you remove this, use the :focus and add some custom visual indicator like box-shadow or border.

3. Too much usage oh heading tags. There are text which should not be wrapped inside heading tags. Like the second text in the hero box, the second text after the "why choose easybank", the four text in the four circles, lastly, the last 4 text on the article sections. Those mentioned should just only been wrapped inside p tags. Because if you made it all heading tags, users who use text-to-speech will have a hard time using your website, thus somehow disabling them.

4. On the article sections, those 4 article should have been inside a tags or their respective title should have been inside a tags, because they are supposed to be links to the article itself.

5. On the footer, social media icons should be wrapped inside as well in a tags because they are social media links. Also the footer text links, need to be inside a tags.

6. On the mobile state, your navbar dropdown hamburger needs to be inside button because this will really help assistive tech, linking them in the control which is the hamburger button is for.

But still, good job on this one ^