Last Shortcut Bugs

src/components/AddShortcut.js

@@ -50,6 +50,14 @@
   return url
 }
 
+const isValidUrl = (urlString) => {
+  try {
+    return Boolean(new URL(urlString))
+  } catch (e) {
+    return false
+  }
+}
+
 const AddShortcut = ({
   onCancel,
   onSave,
@@ -59,6 +67,8 @@
 }) => {
   const [name, setName] = useState(existingName)
   const [url, setUrl] = useState(existingUrl)
+  const [nameError, setNameError] = useState(null)
+  const [urlError, setUrlError] = useState(null)
   useEffect(() => setName(existingName), [existingName])
   useEffect(() => setUrl(existingUrl), [existingUrl])
   const classes = useStyles()
@@ -67,17 +77,49 @@
     setUrl(existingUrl)
     onCancel()
   }
+  const validateName = (newName) => {
+    let newNameError
+    if (newName.length === 0) {
+      newNameError = 'Shortcut name cannot be empty.'
+    } else {
+      newNameError = null
+    }
+    setNameError(newNameError)
+    return newNameError
+  }
+  const validateUrl = (newUrl) => {
+    let newUrlError
+    if (newUrl.length === 0) {
+      newUrlError = 'URL cannot be empty.'
+    } else if (!isValidUrl(addProtocolToURLIfNeeded(newUrl))) {
+      newUrlError = 'URL is not valid.'
+    } else {
+      newUrlError = null
+    }
+    setUrlError(newUrlError)
+    return newUrlError
+  }
+  const validateForm = () => {
+    const newNameError = validateName(name)
+    const newUrlError = validateUrl(url)
+    return newNameError || newUrlError
+  }
   const onSaveClick = () => {
     const newUrl = addProtocolToURLIfNeeded(url)
+    if (validateForm()) {
+      return
+    }
     onSave(existingId, name, newUrl)
     setName(name)
     setUrl(newUrl)
   }
   const changeName = (e) => {
     setName(e.target.value)
+    validateName(e.target.value)
   }
   const changeUrl = (e) => {
     setUrl(e.target.value)
+    validateUrl(e.target.value)
   }
   return (
     <Notification
@@ -100,8 +142,8 @@
               label="Name"
               value={name}
               onChange={changeName}
-              error={name.length === 0}
-              helperText={name.length === 0 && 'Bookmark name cannot be empty.'}
+              error={!!nameError}
+              helperText={nameError}
               className={classes.textField}
             />
             <TextField
@@ -110,8 +152,8 @@
               label="URL"
               value={url}
               onChange={changeUrl}
-              error={url.length === 0}
-              helperText={url.length === 0 && 'URL cannot be empty.'}
+              error={!!urlError}
+              helperText={urlError}
               className={classes.textField}
             />
           </div>
@@ -130,8 +172,8 @@
             onClick={onSaveClick}
             className={classes.yesButton}
             variant="contained"
-            disabled={name.length === 0 || url.length === 0}
             disableElevation
+            disabled={!!(nameError || urlError)}
           >
             Save
           </Button>

src/components/FrontpageShortcutList.js

@@ -36,6 +36,7 @@ const useStyles = makeStyles((theme) => ({
     flexDirection: 'row',
     maxWidth: '550px',
     flexWrap: 'wrap',
+    zIndex: 1.4e3,
   },
   addCircle: {
     marginTop: '24px',
@@ -61,6 +62,7 @@ const useStyles = makeStyles((theme) => ({
   },
   addShortcutWrapper: {
     width: '400px',
+    position: 'relative',
   },
 }))
 const FrontpageShortcutList = ({ openHandler, user }) => {

src/pages/account.js

@@ -376,7 +376,7 @@ const Account = ({ data: fallbackData }) => {
         {localStorageFeaturesManager.getFeatureValue(LAUNCH_BOOKMARKS) !==
           'false' && (
           <AccountItem
-            name="Bookmarks"
+            name="Shortcuts"
             actionButton={
               <Switch
                 checked={bookmarks}

src/pages/index.js

@@ -255,9 +255,6 @@ const useStyles = makeStyles((theme) => ({
     zIndex: 1e4, // must be higher than all content besides ads and modal
     margin: theme.spacing(1),
   },
-  frontpageShortcutList: {
-    zIndex: 1.4e3,
-  },
   logo: {
     height: 50,
     margin: theme.spacing(0.5),
@@ -1143,7 +1140,7 @@ const Index = ({ data: fallbackData, userAgent }) => {
               {localStorageFeaturesManager.getFeatureValue(LAUNCH_BOOKMARKS) !==
                 'false' &&
                 bookmarkWidgetEnabled && (
-                  <div className={classes.frontpageShortcutList}>
+                  <div>
                     <FrontpageShortcutListContainer
                       userId={userId}
                       user={user}