Project Portfolio Kasia

[Kasssiiii]

Netlify link:
https://portfolio-kasia.netlify.app/

Figma design:
https://www.figma.com/design/dXRfvAzJsWDAA1jmDEDQcE/Kasia-Figma-designs?node-id=1078-2022&m=dev

[T-Thiry]

First off, great work on your portfolio! I checked your Lighthouse scores and most of them are hitting 100%, which is great 🎉

A few thoughts and suggestions as I went through your code and site:

Overall Code Quality
I really like that you’re using :root for your CSS variables, that’s a great habit for maintainability and theming. Some of your styled-components are quite large and contain all the breakpoints and layout rules in one place. It might be easier to maintain if you split them into smaller, more focused components. I noticed a few repeated media queries (like .btn a) that could be combined to keep the CSS DRY.

On very small screens (around 320px wide), there’s a bit of horizontal scrolling, it seems like some elements have widths or padding that push the layout past the viewport. You could tweak those for a smoother mobile experience. You’ve got some ARIA labels that might not be needed because the surrounding text already describes the content.

Folder Structure
I like that you’ve separated assets from components. One little improvement could be renaming the projects folder in assets to something like project-images so it’s clear which one has the images and which has the components.

Skills Section
I noticed the hover effect on your skills headings. Since they’re just headings and not clickable, you might consider removing the hover so it doesn’t feel like a button.

Animation
The keyframe dots animation at the bottom is a nice touch! A few of the dots look slightly inconsistent in movement, but it’s nothing major.

Overall, it’s a clean, well-structured project. Just needs a few small tweaks 🙌

[JennieDalgren]

✅