Skip to content

PR Todo-Idgz#48

Open
irisdgz wants to merge 4 commits intoTechnigo:mainfrom
irisdgz:main
Open

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

Conversation

@irisdgz
Copy link
Copy Markdown

@irisdgz irisdgz commented Jan 18, 2026

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

Copy link
Copy Markdown

@Npahlfer Npahlfer left a comment

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