fixed minoe issues

mate-academy · Sep 27, 2023 · f9ab0d5 · f9ab0d5
1 parent 3b013c9
commit f9ab0d5
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 16 deletions.
diff --git a/src/components/TodoApp/TodoApp.tsx b/src/components/TodoApp/TodoApp.tsx
@@ -31,13 +31,23 @@ export const TodoApp: React.FC<Props> = ({
   const handleTodoSave = async (event: React.FormEvent<HTMLFormElement>) => {
     event.preventDefault();
 
-    if (todosTitle) {
-      await onTodoUpdate(todosTitle);
-    } else {
-      await onDelete(todo.id);
+    if (todo.title === todosTitle) {
+      setIsEditing(false);
+
+      return;
     }
 
-    setIsEditing(false);
+    try {
+      if (todosTitle) {
+        await onTodoUpdate(todosTitle);
+      } else {
+        await onDelete(todo.id);
+      }
+
+      setIsEditing(false);
+    // eslint-disable-next-line no-empty
+    } catch (error) {
+    }
   };
 
   const handleTodoTitleChange = (

diff --git a/src/components/TodoHeader/TodoHeader.tsx b/src/components/TodoHeader/TodoHeader.tsx
@@ -7,7 +7,6 @@ type Props = {
   onSubmit: (todo: Todo) => void,
   todo?: Todo | null,
   userId: number,
-  // tempTodo: Todo | null,
   isLoading: boolean,
   errorMessage: string,
   request: boolean;
@@ -69,16 +68,7 @@ export const TodoHeader: React.FC<Props> = ({
 
   return (
     <header className="todoapp__header">
-      {/* {activeTodosCount > 0 && (
-        // eslint-disable-next-line jsx-a11y/control-has-associated-label
-        <button
-          type="button"
-          className={classNames('todoapp__toggle-all', {
-            active: isAllCompleted,
-          })}
-          data-cy="ToggleAllButton"
-        />
-      )} */}
+      {/* {Boolean(todos.lenght) && ( */}
       <button
         type="button"
         aria-label="text"
@@ -88,6 +78,7 @@ export const TodoHeader: React.FC<Props> = ({
         data-cy="ToggleAllButton"
         onClick={onToggleAll}
       />
+      {/* )} */}
 
       <form
         onSubmit={handleSubmit}