From 8c0f3b230491c7f783a67afc16eebf9c1d507ddf Mon Sep 17 00:00:00 2001
From: Philipp Heckel <pheckel@datto.com>
Date: Sat, 19 Feb 2022 22:26:58 -0500
Subject: [PATCH] Add dialog

---
 web/src/AddDialog.js |  55 ++++++++++++++++++++++++
 web/src/App.js       | 100 +++++++++++++------------------------------
 web/src/Title.js     |  17 --------
 web/src/theme.js     |   2 +-
 4 files changed, 86 insertions(+), 88 deletions(-)
 create mode 100644 web/src/AddDialog.js
 delete mode 100644 web/src/Title.js

diff --git a/web/src/AddDialog.js b/web/src/AddDialog.js
new file mode 100644
index 0000000..b8ebe61
--- /dev/null
+++ b/web/src/AddDialog.js
@@ -0,0 +1,55 @@
+import * as React from 'react';
+import Button from '@mui/material/Button';
+import TextField from '@mui/material/TextField';
+import Dialog from '@mui/material/Dialog';
+import DialogActions from '@mui/material/DialogActions';
+import DialogContent from '@mui/material/DialogContent';
+import DialogContentText from '@mui/material/DialogContentText';
+import DialogTitle from '@mui/material/DialogTitle';
+import {useState} from "react";
+import Subscription from "./Subscription";
+
+const defaultBaseUrl = "https://ntfy.sh"
+
+const AddDialog = (props) => {
+    const [topic, setTopic] = useState("");
+    const handleCancel = () => {
+        setTopic('');
+        props.onCancel();
+    }
+    const handleSubmit = () => {
+        const subscription = new Subscription(defaultBaseUrl, topic);
+        props.onSubmit(subscription);
+        setTopic('');
+    }
+    return (
+        <>
+            <Dialog open={props.open} onClose={props.onClose}>
+                <DialogTitle>Subscribe to topic</DialogTitle>
+                <DialogContent>
+                    <DialogContentText>
+                        Topics may not be password-protected, so choose a name that's not easy to guess.
+                        Once subscribed, you can PUT/POST notifications.
+                    </DialogContentText>
+                    <TextField
+                        autoFocus
+                        margin="dense"
+                        id="name"
+                        label="Topic name, e.g. phil_alerts"
+                        value={topic}
+                        onChange={ev => setTopic(ev.target.value)}
+                        type="text"
+                        fullWidth
+                        variant="standard"
+                    />
+                </DialogContent>
+                <DialogActions>
+                    <Button onClick={handleCancel}>Cancel</Button>
+                    <Button onClick={handleSubmit} autoFocus disabled={topic === ""}>Subscribe</Button>
+                </DialogActions>
+            </Dialog>
+        </>
+    );
+};
+
+export default AddDialog;
diff --git a/web/src/App.js b/web/src/App.js
index 76f0594..dfadb1c 100644
--- a/web/src/App.js
+++ b/web/src/App.js
@@ -4,9 +4,8 @@ import Container from '@mui/material/Container';
 import Typography from '@mui/material/Typography';
 import Box from '@mui/material/Box';
 import Link from '@mui/material/Link';
-import Subscription from './Subscription';
 import WsConnection from './WsConnection';
-import {createTheme, styled, ThemeProvider} from '@mui/material/styles';
+import {styled, ThemeProvider} from '@mui/material/styles';
 import CssBaseline from '@mui/material/CssBaseline';
 import MuiDrawer from '@mui/material/Drawer';
 import MuiAppBar from '@mui/material/AppBar';
@@ -17,7 +16,6 @@ import Divider from '@mui/material/Divider';
 import IconButton from '@mui/material/IconButton';
 import Badge from '@mui/material/Badge';
 import Grid from '@mui/material/Grid';
-import Paper from '@mui/material/Paper';
 import MenuIcon from '@mui/icons-material/Menu';
 import ChevronLeftIcon from '@mui/icons-material/ChevronLeft';
 import NotificationsIcon from '@mui/icons-material/Notifications';
@@ -28,19 +26,8 @@ import SettingsIcon from "@mui/icons-material/Settings";
 import AddIcon from "@mui/icons-material/Add";
 import Card from "@mui/material/Card";
 import {Button, CardActions, CardContent, Stack} from "@mui/material";
-
-function Copyright(props) {
-    return (
-        <Typography variant="body2" color="text.secondary" align="center" {...props}>
-            {'Copyright © '}
-            <Link color="inherit" href="https://mui.com/">
-                Your Website
-            </Link>{' '}
-            {new Date().getFullYear()}
-            {'.'}
-        </Typography>
-    );
-}
+import AddDialog from "./AddDialog";
+import theme from "./theme";
 
 const drawerWidth = 240;
 
@@ -88,32 +75,29 @@ const Drawer = styled(MuiDrawer, { shouldForwardProp: (prop) => prop !== 'open'
     }),
 );
 
-const mdTheme = createTheme();
 
 const SubscriptionNav = (props) => {
     const subscriptions = props.subscriptions;
     return (
-        <div className="subscriptionList">
+        <>
             {Object.keys(subscriptions).map(id =>
-                <SubscriptionItem
+                <SubscriptionNavItem
                     key={id}
                     subscription={subscriptions[id]}
                     selected={props.selectedSubscription === subscriptions[id]}
                     onClick={() => props.handleSubscriptionClick(id)}
                 />)
             }
-        </div>
+        </>
     );
 }
 
-const SubscriptionItem = (props) => {
+const SubscriptionNavItem = (props) => {
     const subscription = props.subscription;
     return (
-        <ListItemButton onClick={props.onClick}>
-            <ListItemIcon>
-                <ChatBubbleOutlineIcon />
-            </ListItemIcon>
-            <ListItemText primary={subscription.shortUrl()} />
+        <ListItemButton onClick={props.onClick} selected={props.selected}>
+            <ListItemIcon><ChatBubbleOutlineIcon /></ListItemIcon>
+            <ListItemText primary={subscription.shortUrl()}/>
         </ListItemButton>
     );
 }
@@ -136,68 +120,48 @@ const NotificationItem = (props) => {
                     {notification.time}
                 </Typography>
                 {notification.title && <Typography variant="h5" component="div">
-                    title: {notification.title}
+                    {notification.title}
                 </Typography>}
                 <Typography variant="body1">
-                    msg: {notification.message}
+                    {notification.message}
                 </Typography>
             </CardContent>
-            <CardActions>
-                <Button size="small">Learn More</Button>
-            </CardActions>
         </Card>
     );
 }
 
-const defaultBaseUrl = "https://ntfy.sh"
-
-const SubscriptionAddForm = (props) => {
-    const [topic, setTopic] = useState("");
-    const handleSubmit = (ev) => {
-        ev.preventDefault();
-        props.onSubmit(new Subscription(defaultBaseUrl, topic));
-        setTopic('');
-    }
-    return (
-        <form onSubmit={handleSubmit}>
-            <input
-                type="text"
-                value={topic}
-                onChange={ev => setTopic(ev.target.value)}
-                placeholder="Topic name, e.g. phil_alerts"
-                required
-                />
-        </form>
-    );
-}
-
 const App = () => {
-    const [open, setOpen] = React.useState(true);
+    const [drawerOpen, setDrawerOpen] = useState(true);
     const [subscriptions, setSubscriptions] = useState({});
     const [selectedSubscription, setSelectedSubscription] = useState(null);
     const [connections, setConnections] = useState({});
+    const [addDialogOpen, setAddDialogOpen] = useState(false);
     const subscriptionChanged = (subscription) => {
         setSubscriptions(prev => ({...prev, [subscription.id]: subscription})); // Fake-replace
     };
-    const addSubscription = (subscription) => {
+    const handleAddSubmit = (subscription) => {
+        setAddDialogOpen(false);
         const connection = new WsConnection(subscription, subscriptionChanged);
         setSubscriptions(prev => ({...prev, [subscription.id]: subscription}));
         setConnections(prev => ({...prev, [connection.id]: connection}));
         connection.start();
     };
+    const handleAddCancel = () => {
+        setAddDialogOpen(false);
+    }
     const handleSubscriptionClick = (subscriptionId) => {
         console.log(`handleSubscriptionClick ${subscriptionId}`)
         setSelectedSubscription(subscriptions[subscriptionId]);
     };
     const notifications = (selectedSubscription !== null) ? selectedSubscription.notifications : [];
     const toggleDrawer = () => {
-        setOpen(!open);
+        setDrawerOpen(!drawerOpen);
     };
     return (
-        <ThemeProvider theme={mdTheme}>
+        <ThemeProvider theme={theme}>
             <Box sx={{ display: 'flex' }}>
                 <CssBaseline />
-                <AppBar position="absolute" open={open}>
+                <AppBar position="absolute" open={drawerOpen}>
                     <Toolbar
                         sx={{
                             pr: '24px', // keep right padding when drawer closed
@@ -211,7 +175,7 @@ const App = () => {
                             onClick={toggleDrawer}
                             sx={{
                                 marginRight: '36px',
-                                ...(open && { display: 'none' }),
+                                ...(drawerOpen && { display: 'none' }),
                             }}
                         >
                             <MenuIcon />
@@ -232,7 +196,7 @@ const App = () => {
                         </IconButton>
                     </Toolbar>
                 </AppBar>
-                <Drawer variant="permanent" open={open}>
+                <Drawer variant="permanent" open={drawerOpen}>
                     <Toolbar
                         sx={{
                             display: 'flex',
@@ -259,7 +223,7 @@ const App = () => {
                             </ListItemIcon>
                             <ListItemText primary="Settings" />
                         </ListItemButton>
-                        <ListItemButton>
+                        <ListItemButton onClick={() => setAddDialogOpen(true)}>
                             <ListItemIcon>
                                 <AddIcon />
                             </ListItemIcon>
@@ -281,21 +245,17 @@ const App = () => {
                 >
                     <Toolbar />
                     <Container maxWidth="lg" sx={{ mt: 4, mb: 4 }}>
-
                         <Grid container spacing={3}>
-                            <SubscriptionAddForm onSubmit={addSubscription}/>
                             <NotificationList notifications={notifications}/>
-                            {/* Recent Orders */}
-                            <Grid item xs={12}>
-                                <Paper sx={{ p: 2, display: 'flex', flexDirection: 'column' }}>
-
-                                </Paper>
-                            </Grid>
                         </Grid>
-                        <Copyright sx={{ pt: 4 }} />
                     </Container>
                 </Box>
             </Box>
+            <AddDialog
+                open={addDialogOpen}
+                onCancel={handleAddCancel}
+                onSubmit={handleAddSubmit}
+            />
         </ThemeProvider>
     );
 }
diff --git a/web/src/Title.js b/web/src/Title.js
deleted file mode 100644
index d686a1a..0000000
--- a/web/src/Title.js
+++ /dev/null
@@ -1,17 +0,0 @@
-import * as React from 'react';
-import PropTypes from 'prop-types';
-import Typography from '@mui/material/Typography';
-
-function Title(props) {
-  return (
-    <Typography component="h2" variant="h6" color="primary" gutterBottom>
-      {props.children}
-    </Typography>
-  );
-}
-
-Title.propTypes = {
-  children: PropTypes.node,
-};
-
-export default Title;
diff --git a/web/src/theme.js b/web/src/theme.js
index 4b65d13..d856531 100644
--- a/web/src/theme.js
+++ b/web/src/theme.js
@@ -8,7 +8,7 @@ const theme = createTheme({
       main: '#338574',
     },
     secondary: {
-      main: '#338574',
+      main: '#6cead0',
     },
     error: {
       main: red.A400,