From 96d05e67b227a63373d7a579556e6b69b15dda4e Mon Sep 17 00:00:00 2001
From: Colin <colin.diesh@gmail.com>
Date: Mon, 30 Aug 2021 13:46:46 -0400
Subject: [PATCH] [desktop beta] Make the default session name get a timestamp
 and make filter not shown by default

---
 .../StartScreen/PreloadedDatasetSelector.tsx  | 34 +++++++++++++------
 .../src/StartScreen/preloadedConfigs.js       |  4 ++-
 2 files changed, 27 insertions(+), 11 deletions(-)

diff --git a/products/jbrowse-desktop/src/StartScreen/PreloadedDatasetSelector.tsx b/products/jbrowse-desktop/src/StartScreen/PreloadedDatasetSelector.tsx
index 33a698fba9f..68a5e66b456 100644
--- a/products/jbrowse-desktop/src/StartScreen/PreloadedDatasetSelector.tsx
+++ b/products/jbrowse-desktop/src/StartScreen/PreloadedDatasetSelector.tsx
@@ -3,11 +3,14 @@ import {
   Button,
   Checkbox,
   FormControlLabel,
+  IconButton,
   TextField,
+  Tooltip,
   Typography,
   makeStyles,
 } from '@material-ui/core'
 import PluginManager from '@jbrowse/core/PluginManager'
+import SearchIcon from '@material-ui/icons/Search'
 import { createPluginManager } from './util'
 import preloadedConfigs from './preloadedConfigs'
 import deepmerge from 'deepmerge'
@@ -37,22 +40,31 @@ function PreloadedDatasetSelector({
   const classes = useStyles()
   const [selected, setSelected] = useState({} as Record<string, boolean>)
   const [search, setSearch] = useState('')
+  const [filterShown, setFilterShown] = useState(false)
 
   return (
     <div className={classes.container}>
       <div style={{ display: 'flex' }}>
         <Typography className={classes.header}>
           Select assembly (or multiple assemblies) for your session
+          <IconButton onClick={() => setFilterShown(!filterShown)}>
+            <Tooltip title="Show filter?">
+              <SearchIcon />
+            </Tooltip>
+          </IconButton>
         </Typography>
         <Button
           className={classes.button}
           onClick={async () => {
-            const pm = await createPluginManager(
-              deepmerge.all(
-                // @ts-ignore
-                Object.keys(selected).map(name => preloadedConfigs[name]),
-              ),
+            const config = deepmerge.all(
+              // @ts-ignore
+              Object.keys(selected).map(name => preloadedConfigs[name]),
             )
+
+            // @ts-ignore
+            config.defaultSession.name +=
+              ' ' + new Date().toLocaleString('en-US')
+            const pm = await createPluginManager(config)
             setPluginManager(pm)
           }}
           variant="contained"
@@ -61,11 +73,13 @@ function PreloadedDatasetSelector({
           Go
         </Button>
       </div>
-      <TextField
-        label="Filter datasets"
-        value={search}
-        onChange={event => setSearch(event.target.value as string)}
-      />
+      {filterShown ? (
+        <TextField
+          label="Filter datasets"
+          value={search}
+          onChange={event => setSearch(event.target.value as string)}
+        />
+      ) : null}
       <br />
       {Object.keys(preloadedConfigs)
         .filter(name => (search ? name.match(new RegExp(search, 'i')) : true))
diff --git a/products/jbrowse-desktop/src/StartScreen/preloadedConfigs.js b/products/jbrowse-desktop/src/StartScreen/preloadedConfigs.js
index 37bc843590c..e7759fcdd22 100644
--- a/products/jbrowse-desktop/src/StartScreen/preloadedConfigs.js
+++ b/products/jbrowse-desktop/src/StartScreen/preloadedConfigs.js
@@ -1,4 +1,4 @@
-export default {
+const preloadedConfigs = {
   hg38: {
     assemblies: [
       {
@@ -423,3 +423,5 @@ export default {
     ],
   },
 }
+
+export default preloadedConfigs