added improvements

mate-academy · Jul 19, 2023 · a436864 · a436864
1 parent 8405779
commit a436864
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -153,7 +153,7 @@ export const App: React.FC = () => {
                     </div>
                   )}
 
-                {userPosts.length > 0 && (
+                {userPosts.length > 0 && !isLoading && (
                   <PostsList
                     userPosts={userPosts}
                     handleSelectPost={handleSelectPost}

diff --git a/src/components/NewCommentForm.tsx b/src/components/NewCommentForm.tsx
@@ -54,7 +54,8 @@ export const NewCommentForm: React.FC<Props> = ({
       setIsCommentTextError(false);
     }
 
-    if (name.trim().length
+    if (
+      name.trim().length
       && email.trim().length
       && commentText.trim().length
     ) {
@@ -88,7 +89,10 @@ export const NewCommentForm: React.FC<Props> = ({
               'is-danger': isNameError,
             })}
             value={name}
-            onChange={(e) => setName(e.target.value)}
+            onChange={(e) => {
+              setName(e.target.value);
+              setIsNameError(false);
+            }}
           />
 
           <span className="icon is-small is-left">
@@ -119,15 +123,18 @@ export const NewCommentForm: React.FC<Props> = ({
 
         <div className="control has-icons-left has-icons-right">
           <input
-            type="text"
+            type="email"
             name="email"
             id="comment-author-email"
             placeholder="email@test.com"
             className={cn('input', {
               'is-danger': isEmailError,
             })}
             value={email}
-            onChange={(e) => setEmail(e.target.value)}
+            onChange={(e) => {
+              setEmail(e.target.value);
+              setIsEmailError(false);
+            }}
           />
 
           <span className="icon is-small is-left">
@@ -165,7 +172,10 @@ export const NewCommentForm: React.FC<Props> = ({
               'is-danger': isCommentTextError,
             })}
             value={commentText}
-            onChange={(e) => setCommentText(e.target.value)}
+            onChange={(e) => {
+              setCommentText(e.target.value);
+              setIsCommentTextError(false);
+            }}
           />
         </div>