fixed navbar warning

Author: archies-coder

client/src/components/add-student/AddStudent.js

@@ -179,6 +179,7 @@ export default class AddStudent extends Component {
                                     value={this.state.date}
                                 />
                             </Modal>
+                            
                         </div>
                     )}
                 </AuthContext.Consumer>

client/src/components/fees/Fee.js

@@ -25,7 +25,7 @@ const Fee = (props) => {
 
     useEffect(()=> getStudents(),[])
 
-    const getStudents= ()=>{
+    const getStudents = ()=>{
         setLoading(true)
         fetch('/api/students',{
             method: 'GET',
@@ -133,9 +133,8 @@ const Fee = (props) => {
 
     const {fees} = selectedStudent;
 
-    return (loading) ?<div className='container spinner-container'>
-            <MyLoader loading={loading} />
-        </div>:
+    return (loading) ?
+            <MyLoader loading={loading} />:
         <div className="container">
             <h1 className='main-heading'>Fees</h1>
             <div className="row">

client/src/components/navigation/AppNav.js

@@ -31,70 +31,32 @@ export default class AppNavbar extends Component {
           <Navbar color='dark' dark expand='md'>
             {context.state.isAuth && <Redirect from='/login' to='/' />}
             {!context.state.isAuth && <Redirect from='/' to='/login' />}
-            <NavbarBrand>
-              <Link to='/'>
+            <NavbarBrand tag={Link} to='/'>
                 <span className='navbar-brand'>NitMoi</span>
-              </Link>
             </NavbarBrand>
             <NavbarToggler onClick={this.toggle} />
             <Collapse isOpen={this.state.isOpen} navbar>
             <Nav className="mr-auto" navbar>
-              <NavItem>
-                  <React.Fragment>
-                    <NavLink>
-                      <Link to='/list' className='nav-link'>
-                        Students
-                      </Link>
-                    </NavLink>
-                  </React.Fragment>
-              </NavItem>
-              <NavItem>
-                <React.Fragment>
-                    <NavLink>
-                      <Link to='/add' className='nav-link'>
-                        Add
-                      </Link>
-                    </NavLink>
-                </React.Fragment>
-              </NavItem>
-              <NavItem>
-                <React.Fragment>
-                    <NavLink>
-                      <Link to='/attendance' className='nav-link'>
-                        Attendance
-                      </Link>
-                    </NavLink>
-                </React.Fragment>
-              </NavItem>
-              <NavItem>
-                <React.Fragment>
-                  <NavLink>
-                    <Link to='/fees' className='nav-link'>
-                      Fee
-                    </Link>
-                  </NavLink>
-                </React.Fragment>
-              </NavItem>
+                <NavLink tag={Link} to='/list' className='nav-link'>
+                  Students
+                </NavLink>
+                <NavLink tag={Link} to='/add' className='nav-link'>
+                  Add
+                </NavLink>
+                <NavLink tag={Link} to='/attendance' className='nav-link'>
+                  Attendance
+                </NavLink>
+                <NavLink tag={Link} to='/fees' className='nav-link'>
+                  Fee
+                </NavLink>
             </Nav>
             <Nav className="ml-auto" navbar>
-            <NavItem>
-              <React.Fragment>
-                <NavLink>
-                {!context.state.isAuth&&<Link to='/login' className='nav-link'>
-                    Login
-                  </Link>}
-                </NavLink>
-              </React.Fragment>
-              </NavItem>
-                <NavItem>
-                  <React.Fragment>
-                    <NavLink>
-                      {context.state.isAuth && <Link to='/logout' className='nav-link' onClick={context.handleLogout}>
-                        Logout
-                      </Link>}
-                    </NavLink>
-                  </React.Fragment>
-                </NavItem>
+                {!context.state.isAuth && <NavLink tag={Link} to='/login' className='nav-link'>
+                  Login
+                </NavLink>}
+                {context.state.isAuth && <NavLink tag={Link} to='/logout' className='nav-link'>
+                  Logout
+                </NavLink>}
             </Nav>
           </Collapse>
           </Navbar>

client/src/components/student-list/Student.css

@@ -39,7 +39,7 @@
 }
 
 @media screen and (min-width: 1px) and (max-width: 767px){
-    .stud-card {
+    .stud-card{
         min-width: 350px;
     }
 }