Develop

[an-marryKyslenko]

DEMO LINK

[volodymyr-soltys97]

Good job 👍
Let's improve your code

1. When we delete all todos using the clear button, the loader is shown everywhere, we need to fix this and show only those todos that are deleted

2. It is a bad practice to write all the code in one component, you need to separate the logic into different components, for example Header, Footer, TodoItem, TodoList

[volodymyr-soltys97]

Hi, the comments from the previous review still not fixed, if you need help feel free in the fe_chat

[volodymyr-soltys97]

Don't need to skip test

Suggested change

	it.skip('should cancel if title was not changed', () => {
	it('should cancel if title was not changed', () => {

[an-marryKyslenko]

I skiped one test becouse mentor from chat told me to do this.

[volodymyr-soltys97]

Almost done!

[volodymyr-soltys97]

It is important to follow the same way of creating functions everywhere, fix it everywhere

Suggested change

	function editTodo(data: TodoTitleOrCompleted, todoId: number) {
	сщтіе editTodo = (data: TodoTitleOrCompleted, todoId: number) => {

[volodymyr-soltys97]

Remove all comments

Suggested change

	// #endregion
	// #region edit Todo