Changes

Hi @iambenkis, good job so far!

• Please refrain from using !important in your CSS (at line 133) as it is often considered bad practice because it has side effects that mess with one of CSS's core mechanisms: specificity.
• It makes your CSS harder to maintain and debug on the long run.

Almost there 🤞

• please change the color of the :hover effect in your CSS to match the one in the Figma template

Hi @iambenkis 👋, You did an excellent job 👍! Your website looks nice and clean according to the Figma template. Here are a few suggestions which can improve your website user experience:

• Try adding a background to the navbar so that the text in the navbar does not overlap with the rest of the website.
• You can put the whole contact form in one div to make it more responsive.
• Try giving a different class name to the header section.
• Try adding your social links to the social media icons.

Welldone @iambenkis 🤞'

Kindly change the laguage tag text under work section (html, css, javascript) to lower case. This will help you achieve exact replica of the design you are working with.

Blossom Aniedi

• Great use of HTML semantic tags but it would be advisable to use other semantic tags like section, and article alongside divs.
• The alternative images should always be specified.

Hi @iambenkis, 👋

Lots of bugs were found inside the .html file but few in the .js file (on the locale-storage feature branch)
The only bug that was detected in the script.js file which might cause a linter error is:

1. On line 125 of scripit.js, there is a trailing space. Please remove it to fix the linter error.