Skip to content

Conversation

cayalame
Copy link
Contributor

@cayalame cayalame commented Dec 3, 2020

worked on login, register, navbar

super(props);
}

// parser(input) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove all comments

@cayalame cayalame closed this Dec 3, 2020



export const Wrapper = styled.body`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change styled.body to styled.div

font-weight: bold;
`;


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix your spacing

::placeholder {
font-family: 'Trebuchet MS', sans-serif;
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spacing

text-align: left;
font-family: 'Trebuchet MS', sans-serif;
width: 200px;
padding: 10px 10px; //inside padding
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comment



export const Button = styled.button`
/* This renders the buttons above... Edit me! */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comment

text-decoration: none;
height: 100%;
padding: 0px 20px;

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make sure spacing like this is everywhere before a :hover or &something

{auth.isAuthenticated ? (
<>
<button onClick={logoutUser}>Sign out</button>
<p key={3}>{`Logged in as: You`}</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why key?

<img src={logo}/>
</NavLogo>
<Header>Cloud Haven</Header>
<NavItemLocation>
<>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dont need empty <>

import Home from '../pages/Home/Home';
import PrivateRoute from '../components/private-route/PrivateRoute';
import NewPage from '../components/NewPage/NewPage';


// Check for token to keep user logged in
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove da comments

// Redirect to login
window.location.href = './login';
// Redirect to NewPage, was login
window.location.href = './';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove .

@joesobo joesobo reopened this Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants