Retky

• The code is well organized and clear. 👍
• I think there is missing a design between the mobile version (min: 480px) and desktop version (992px) because when resizing the content display without style.

Tadesse

• The mobile and desktop version is perfect. I love how you fit the styles. 👍
• Design a responsive page for screen size between 480px to 992px. the style breaks on those screens. and I think that got something to do with the max-width property.
• It would be better to have one images folder inside the project and organize different types of images only within that folder.
• it would be good to have a separate ID for each section. since we won't have a repeated section.

Chidiebere

Good points 👏

• Good HTML semantics 💪.
• HTML code looks clean.
• Site is very responsive.
• Sections were clearly defined.
• CSS code also looks very clean and readable for humans 🤓.
• Nice use of comments in your HTML code.

To take note 📝

• There is some indentation that could be fixed so it looks cleaner. (line 183).
• Suggest that use more descriptive class names. Eg. The cards could use a class of "cards". 📝
• Line 273 hard coded whitespace could be styled with padding or margin.
• Suggest that the break-point for the media query for mobile should not be 992px rather 600px or less.
• Could be more descriptive on the README.md file by explaining how to clone the repo, as well as mentioning the handlers.
• Try not to leave commented code when presenting.
• In the cards suggest not using buttons for the tech tags.
• Should have a .gitignore file.
• Your title could be more descriptive so it has more meaning.
• It could help you get more organized by having the image assets in one folder rather than having many folders. 📂
• Alts in images could be more descriptive for better accessibility.
• Could help review git flow

Code was good and well organized @oshanedesign

The code is clean and the portfolio looks great @AbrahaKahsay

The code is easy to understand, well organized, project meets the requirement.. However I suggest using classes in one style in order to not repeat code and make it shorter.. @Jord4N-AC

In this issue;

-We found a bug in the javascript of the popup window code;

• There was a point where the code was referenced before it was loaded and this prevented the referenced code from displaying the required content when run.

• This was fixed by placing the referenced code before the one that is referring to it.