From 9d7131f8dbffb5934b5db9e19181ee64c6f50bcf Mon Sep 17 00:00:00 2001 From: wrigh393 Date: Tue, 29 Jun 2021 16:56:12 -0400 Subject: [PATCH 1/2] Changed Autocomplete loading prop value to be same as loading variable value; Changed noOptions text to be more specific --- src/components/QueueSelector/QueueSelector.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/components/QueueSelector/QueueSelector.js b/src/components/QueueSelector/QueueSelector.js index 072471d..5a4c41f 100644 --- a/src/components/QueueSelector/QueueSelector.js +++ b/src/components/QueueSelector/QueueSelector.js @@ -39,11 +39,11 @@ export default function QueueSelector({ open, setOpen, value, setValue }) { const theme = useTheme(); const useStyles = makeStyles({ - Paper_root: { + Paper_root: { margin: theme.spacing(1) } - }); - const classes = useStyles(); + }); + const classes = useStyles(); // Prepopulate Active Queues from Cookies useEffect( _ => { @@ -155,11 +155,12 @@ export default function QueueSelector({ open, setOpen, value, setValue }) { getOptionLabel={(option) => `${option.name} (${option.number_of_items})`} renderOption={optionRenderer} getOptionSelected={ (option, value) => option.name === value.name } + noOptionsText={"Queue does not exist"} size="small" open={open} onOpen={_ => setOpen(true)} onClose={_ => setOpen(false)} - loading={true} + loading={loading} disableCloseOnSelect disableListWrap fullWidth From 4cf1888cd2a6eb4093bfe64de176a7bcf17dd014 Mon Sep 17 00:00:00 2001 From: Justin Campbell Date: Thu, 8 Jul 2021 17:20:01 -0400 Subject: [PATCH 2/2] Modify noOptionsText to indicate what queue does not exist --- src/components/QueueSelector/QueueSelector.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/components/QueueSelector/QueueSelector.js b/src/components/QueueSelector/QueueSelector.js index 5a4c41f..f46df4b 100644 --- a/src/components/QueueSelector/QueueSelector.js +++ b/src/components/QueueSelector/QueueSelector.js @@ -31,6 +31,7 @@ const getQueueCounts = async (access_token) => { export default function QueueSelector({ open, setOpen, value, setValue }) { const [queueCounts, setQueueCounts] = useState([]); const [isFirstRender, setIsFirstRender] = useState(true); + const [searchValue, setSearchValue] = useState(""); const access_token = useToken(); const loading = open && queueCounts.length === 0; @@ -152,10 +153,11 @@ export default function QueueSelector({ open, setOpen, value, setValue }) { options={queueCounts} value={value} onChange={handleChange} + onInputChange={(event, value, reason) => setSearchValue(value)} getOptionLabel={(option) => `${option.name} (${option.number_of_items})`} renderOption={optionRenderer} getOptionSelected={ (option, value) => option.name === value.name } - noOptionsText={"Queue does not exist"} + noOptionsText={`'${searchValue}' queue does not exist.`} size="small" open={open} onOpen={_ => setOpen(true)}