Skip to content

Conversation

@LeonEkelund
Copy link

No description provided.

Copy link

@fridascript fridascript left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heyhey Leon great job, it looks good and the filtering works very well!!
I left a little heads up comment regarding the width of the page- apart from that little bugg it looks splendid! ✨

@JennieDalgren JennieDalgren self-assigned this Oct 26, 2025
Copy link

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job with this project!
You have met all requreiments but the responsiveness.
Fix that and let me know when it's done so I can approve this project.

All in all, your code is well structured and easy to follow. You could practise a bit more on the css and see if you can refactor that a bit. Find more similarities in the css and avoid repeating your self. You can use more variables also for sizes that you use a lot like padding, margin, gaps etc.

Keep up the good work!

Copy link

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants