refactor: apply PR requested changes

graasp · Nov 18, 2020 · 85c70bd · 85c70bd
1 parent 6d1cbc3
commit 85c70bd
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 9 deletions.
diff --git a/scripts/setup.js b/scripts/setup.js
@@ -16,8 +16,6 @@ const env = JSON.stringify({
   GOOGLE_API_KEY,
   GOOGLE_ANALYTICS_ID,
   LOGGING_LEVEL,
-  HELLO: 'er',
-  REACT_APP_HELLO_D: 'wef',
 });
 
 fs.writeFileSync(path.join(DEFAULT_PATH, NAME), env, { encoding: 'utf8' });
diff --git a/src/components/common/WelcomeContent.js b/src/components/common/WelcomeContent.js
@@ -17,7 +17,7 @@ import {
 } from '../../config/paths';
 import { VISIT_SPACE_BUTTON_CLASS } from '../../config/selectors';
 
-const styles = theme => ({
+const styles = (theme) => ({
   ...Styles(theme),
   wrapper: {
     display: 'flex',
@@ -102,6 +102,7 @@ class WelcomeContent extends Component {
           {t('Welcome to Graasp Desktop')}
         </Typography>
         <ButtonGroup
+          disableElevation
           variant="contained"
           color="primary"
           aria-label="contained group primary button"

diff --git a/src/components/dashboard/ActionBarChart.js b/src/components/dashboard/ActionBarChart.js
@@ -35,7 +35,7 @@ class ActionBarChart extends PureComponent {
     theme: PropTypes.shape({
       palette: PropTypes.shape({
         type: PropTypes.string.isRequired,
-        primary: PropTypes.shape().isRequired,
+        primary: PropTypes.shape({}).isRequired,
       }).isRequired,
     }).isRequired,
     i18n: PropTypes.shape({

diff --git a/src/components/dashboard/ActionLineChart.js b/src/components/dashboard/ActionLineChart.js
@@ -35,7 +35,7 @@ class ActionLineChart extends PureComponent {
     theme: PropTypes.shape({
       palette: PropTypes.shape({
         type: PropTypes.string.isRequired,
-        primary: PropTypes.shape().isRequired,
+        primary: PropTypes.shape({}).isRequired,
       }).isRequired,
     }).isRequired,
     i18n: PropTypes.shape({

diff --git a/src/components/signin/SignInScreen.js b/src/components/signin/SignInScreen.js
@@ -51,9 +51,9 @@ const CssTextField = withStyles({
   },
 })(TextField);
 
-const styles = theme => ({
+const styles = (theme) => ({
   ...Styles(theme),
-  graaspLogo: { padding: '0px 45px', width: '95%' },
+  graaspLogo: { padding: theme.spacing(2), width: '95%' },
 });
 
 class SignInScreen extends Component {
@@ -151,13 +151,13 @@ class SignInScreen extends Component {
     dispatchSignIn({ anonymous: true, lang });
   };
 
-  handleUsername = event => {
+  handleUsername = (event) => {
     const username = event.target ? event.target.value : event;
     const usernameIsEmpty = !username.length;
     this.setState({ username, usernameIsEmpty });
   };
 
-  handleKeyPressed = event => {
+  handleKeyPressed = (event) => {
     if (event.key === 'Enter') {
       this.handleSignIn();
     }