Skip to content

context#1266

Open
DebHtm wants to merge 3 commits intomate-academy:masterfrom
DebHtm:develop
Open

context#1266
DebHtm wants to merge 3 commits intomate-academy:masterfrom
DebHtm:develop

Conversation

@DebHtm
Copy link

@DebHtm DebHtm commented Jan 29, 2026

  • Install the Prettier Extension and use these VSCode settings to enable format on save.
  • Implement a solution following the React task guidelines.
  • Use the React TypeScript cheat sheet.
  • Open another terminal and run tests with npm test to ensure your solution is correct.
  • Replace <your_account> with your GitHub username in the DEMO LINK and add it to the PR description.

Copy link

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Almost dome, let`s fix few things:

  1. There should be the error message when i try to submit empty todo
  2. Fix the UI focus of input field.

@DebHtm DebHtm requested a review from brespect January 30, 2026 12:05
Copy link

@Anton-Kuchmasov Anton-Kuchmasov left a comment

Choose a reason for hiding this comment

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

Hey, looks better, but would you mind fixing this?

Image

reference:

Image

you can re-use styles from react_todo-app-with-api (link) or create something up to you
but please, show ErrorMessage UNDER list of todos

Copy link

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

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.

4 participants