Project4 Codereview-yao

client/src/assets/styles/Landing.css

@@ -66,6 +66,8 @@ body {
   text-align: center;
   transition: all 0.15s;
 }
+
+/* Really like the hover effect and active effect that you added to your buttons, they are really beautiful */
 .bookBtn:hover {
   border-color: #fcfbfb;
   transform: scale(1.05, 1.05);

client/src/components/ManageBook.js

@@ -83,6 +83,7 @@ function ManageBook() {
       <button
         className="deleteBtnBook"
         onClick={() => {
+//       Good to know that I can use this window.confirm thing to handle the confirmation for delete actions
           const confirmBox = window.confirm("Are you sure you want to cancel?");
           if (confirmBox === true) {
             removeClass(date, time, tutor);

client/src/components/Register.js

@@ -47,7 +47,7 @@ function Register() {
     }));
     validateInput(evt);
   };
-
+// This is a very good design that you validated the user's password, this is a good practice and our group should add this function too.
   const validateInput = (evt) => {
     let { name, value } = evt.target;
     setError((prev) => {
@@ -62,6 +62,8 @@ function Register() {
   const handleRegister = () => {
     navigate("/login", { replace: true });
   };
+
+//   The align of the components still needs more adjustment
 
   return (
     <main className="card" id="signupCard">

client/src/pages/Landing.js

@@ -10,6 +10,7 @@ function Landing() {
     <>
       <Navbar />
       <div className="mainContainer" role="main">
+//     The landing page is good, looks like a page of a real business
         <div className="banner">
           <div className="subContainer">
             <h1>Tutor Match helps you get the help you need</h1>