-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Google Oauth button redesigned and refactored the button component #13
base: develop
Are you sure you want to change the base?
Conversation
import React from 'react' | ||
import styles from './ui.module.css' | ||
import GoogleLogo from '../../../images/g.png' | ||
export default function GoogleButton({ onClick, textLabel }) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add line above
selectedOption: option, | ||
}) | ||
} | ||
render() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add line above
} | ||
|
||
toggleRadioCheck = (e) => { | ||
var option = e.target.value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use const
toggleRadioCheck = (e) => { | ||
var option = e.target.value | ||
this.setState({ | ||
selectedOption: option, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or actually no variable needed. Just do:
selectedOption: e.target.value
import Dropdown from './dropdown' | ||
import styles from './paragraphsCard.module.css' | ||
|
||
const labelOrientation = 'vertical', | ||
dropdownData = ['All Time', 'Weekly', 'Monthly'], | ||
const dropdownData = ['All Time', 'Weekly', 'Monthly'], | ||
labelData = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Declare labelData in a separate const
}, | ||
] | ||
const dropdownOptions = ['All', 'Practise', 'Arena'] | ||
// labelData = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this commented? If it's not used delete it
// console.log('Given Name: ' + profile.getGivenName()) | ||
// console.log('Family Name: ' + profile.getFamilyName()) | ||
// console.log('Image URL: ' + profile.getImageUrl()) | ||
// console.log('Email: ' + profile.getEmail()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete comments
{/* <div id="customBtn" className="customGPlusSignIn" onClick={this.onSignIn}> | ||
<span className="icon"><img className="googleLogo" src={GoogleLogo} alt="glogo"></img></span> | ||
<span className="buttonText">Sign Up With Google</span> | ||
</div> */} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete
No description provided.