Use colors from theme palette

Sascha created

- Use theme colors for title input.
- Remove inputTitle classes as they are not applied to the TextField.
  This will lead to double borders and artifacts at the field corners.

Change summary

webui/src/components/BugTitleForm/.BugTitleForm.tsx.swp |  0 
webui/src/components/BugTitleForm/BugTitleForm.tsx      | 24 +---------
webui/src/pages/list/ListQuery.tsx                      | 11 +---
webui/src/pages/new/NewBugPage.tsx                      | 23 +---------
webui/src/themes/DefaultDark.ts                         |  5 ++
webui/src/themes/DefaultLight.ts                        |  4 +
6 files changed, 19 insertions(+), 48 deletions(-)

Detailed changes

webui/src/components/BugTitleForm/BugTitleForm.tsx 🔗

@@ -45,26 +45,9 @@ const useStyles = makeStyles((theme) => ({
     marginLeft: theme.spacing(2),
   },
   greenButton: {
-    marginLeft: '8px',
-    backgroundColor: '#2ea44fd9',
-    color: '#fff',
-    '&:hover': {
-      backgroundColor: '#2ea44f',
-    },
-  },
-  titleInput: {
-    borderRadius: theme.shape.borderRadius,
-    borderColor: fade(theme.palette.primary.main, 0.2),
-    borderStyle: 'solid',
-    borderWidth: '1px',
-    backgroundColor: fade(theme.palette.primary.main, 0.05),
-    padding: theme.spacing(0, 0),
-    minWidth: 336,
-    transition: theme.transitions.create([
-      'width',
-      'borderColor',
-      'backgroundColor',
-    ]),
+    marginLeft: theme.spacing(1),
+    backgroundColor: theme.palette.success.main,
+    color: theme.palette.success.contrastText,
   },
 }));
 
@@ -126,7 +109,6 @@ function BugTitleForm({ bug }: Props) {
           inputRef={(node) => {
             issueTitleInput = node;
           }}
-          className={classes.titleInput}
           variant="outlined"
           fullWidth
           margin="dense"

webui/src/pages/list/ListQuery.tsx 🔗

@@ -112,12 +112,9 @@ const useStyles = makeStyles<Theme, StylesProps>((theme) => ({
       padding: theme.spacing(2, 3),
     },
   },
-  greenButton: {
-    backgroundColor: '#2ea44fd9',
-    color: '#fff',
-    '&:hover': {
-      backgroundColor: '#2ea44f',
-    },
+  newIssueButton: {
+    backgroundColor: theme.palette.success.main,
+    color: theme.palette.success.contrastText,
   },
 }));
 
@@ -318,7 +315,7 @@ function ListQuery() {
         <IfLoggedIn>
           {() => (
             <Button
-              className={classes.greenButton}
+              className={classes.newIssueButton}
               variant="contained"
               href="/new"
             >

webui/src/pages/new/NewBugPage.tsx 🔗

@@ -3,7 +3,7 @@ import React, { FormEvent, useState } from 'react';
 import { Button } from '@material-ui/core';
 import Paper from '@material-ui/core/Paper';
 import TextField from '@material-ui/core/TextField/TextField';
-import { fade, makeStyles, Theme } from '@material-ui/core/styles';
+import { makeStyles, Theme } from '@material-ui/core/styles';
 
 import CommentInput from '../../components/CommentInput/CommentInput';
 
@@ -21,19 +21,6 @@ const useStyles = makeStyles((theme: Theme) => ({
     padding: theme.spacing(2),
     overflow: 'hidden',
   },
-  titleInput: {
-    borderRadius: theme.shape.borderRadius,
-    borderColor: fade(theme.palette.primary.main, 0.2),
-    borderStyle: 'solid',
-    borderWidth: '1px',
-    backgroundColor: fade(theme.palette.primary.main, 0.05),
-    padding: theme.spacing(0, 0),
-    transition: theme.transitions.create([
-      'width',
-      'borderColor',
-      'backgroundColor',
-    ]),
-  },
   form: {
     display: 'flex',
     flexDirection: 'column',
@@ -43,11 +30,8 @@ const useStyles = makeStyles((theme: Theme) => ({
     justifyContent: 'flex-end',
   },
   greenButton: {
-    backgroundColor: '#2ea44fd9',
-    color: '#fff',
-    '&:hover': {
-      backgroundColor: '#2ea44f',
-    },
+    backgroundColor: theme.palette.success.main,
+    color: theme.palette.success.contrastText,
   },
 }));
 
@@ -90,7 +74,6 @@ function NewBugPage() {
             issueTitleInput = node;
           }}
           label="Title"
-          className={classes.titleInput}
           variant="outlined"
           fullWidth
           margin="dense"

webui/src/themes/DefaultDark.ts 🔗

@@ -15,6 +15,11 @@ const defaultDarkTheme = createMuiTheme({
       main: '#2a393e',
     },
   },
+  typography: {
+    button: {
+      textTransform: 'none',
+    },
+  },
 });
 
 export default defaultDarkTheme;

webui/src/themes/DefaultLight.ts 🔗

@@ -10,6 +10,10 @@ const defaultLightTheme = createMuiTheme({
     info: {
       main: '#e2f1ff',
     },
+    success: {
+      main: '#2ea44fd9',
+      contrastText: '#fff',
+    },
     text: {
       secondary: '#555',
     },