DGNum/demarches-normaliennes
14
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Merge pull request #10586 from tchak/combo-distinct-values

Browse source 
fix(combobox): allow non unique lists of options
This commit is contained in:
Paul Chavard 2024-07-08 09:43:15 +00:00 committed by GitHub
commit 09db22f81c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
app/javascript/components/react-aria/hooks.ts
View file

@ -56,7 +56,10 @@ export function useSingleList({
onChange?: (item: Item | null) => void;
}) {
const [selectedKey, setSelectedKey] = useState(defaultSelectedKey);
const items = useMemo(() => defaultItems || [], [defaultItems]);
const items = useMemo(
() => (defaultItems ? distinctBy(defaultItems, 'value') : []),
[defaultItems]
);
const selectedItem = useMemo(
() => items.find((item) => item.value == selectedKey) ?? null,
[items, selectedKey]
@ -157,7 +160,10 @@ export function useMultiList({
() => new Set(defaultSelectedKeys ?? [])
);
const [inputValue, setInputValue] = useState('');
const items = useMemo(() => defaultItems || [], [defaultItems]);
const items = useMemo(
() => (defaultItems ? distinctBy(defaultItems, 'value') : []),
[defaultItems]
);
const itemsIndex = useMemo(() => {
const index = new Map<string, Item>();
for (const item of items) {
@ -473,3 +479,8 @@ export function useOnFormReset(onReset?: () => void) {
return ref;
}
function distinctBy<T>(array: T[], key: keyof T): T[] {
const keys = array.map((item) => item[key]);
return array.filter((item, index) => keys.indexOf(item[key]) == index);
}