Skip to content

PR Todo-Idgz - #48

Open
irisdgz wants to merge 4 commits into
Technigo:mainfrom
irisdgz:main
Open

PR Todo-Idgz#48
irisdgz wants to merge 4 commits into
Technigo:mainfrom
irisdgz:main

Conversation

@irisdgz

@irisdgz irisdgz commented Jan 18, 2026

Copy link
Copy Markdown

Please include your Netlify link here. https://iristodolist.netlify.app/

@Npahlfer Npahlfer left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done and good job with your todo app Iris! Your app looks clean is well structured.

I did notice some stuff that could improve it a bit, in the TaskStyles component you reference var(--header-font) for the form inputs. That variable isn’t defined, so the font‑family falls back to the browser default, but it works so no biggie! :)

There is also some inline styling, try to keep inline styling to a minimum and only use it for dynamic styling that changes a lot, And remember that styled components has $props that you can utilize along with the theme for reused colors and such :).

But overall really good job!

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.

2 participants