mirror of
https://github.com/docmost/docmost.git
synced 2026-05-07 06:23:06 +08:00
fix
This commit is contained in:
@@ -1,4 +1,4 @@
|
|||||||
import { useState, useRef, useEffect, useCallback } from "react";
|
import { useState, useRef, useEffect, useCallback, useMemo } from "react";
|
||||||
import { Popover, TextInput } from "@mantine/core";
|
import { Popover, TextInput } from "@mantine/core";
|
||||||
import {
|
import {
|
||||||
IBaseProperty,
|
IBaseProperty,
|
||||||
@@ -6,8 +6,15 @@ import {
|
|||||||
Choice,
|
Choice,
|
||||||
} from "@/features/base/types/base.types";
|
} from "@/features/base/types/base.types";
|
||||||
import { choiceColor } from "@/features/base/components/cells/choice-color";
|
import { choiceColor } from "@/features/base/components/cells/choice-color";
|
||||||
|
import { useUpdatePropertyMutation } from "@/features/base/queries/base-property-query";
|
||||||
|
import { v7 as uuid7 } from "uuid";
|
||||||
import cellClasses from "@/features/base/styles/cells.module.css";
|
import cellClasses from "@/features/base/styles/cells.module.css";
|
||||||
|
|
||||||
|
const CHOICE_COLORS = [
|
||||||
|
"gray", "red", "pink", "grape", "violet", "indigo",
|
||||||
|
"blue", "cyan", "teal", "green", "lime", "yellow", "orange",
|
||||||
|
];
|
||||||
|
|
||||||
type CellMultiSelectProps = {
|
type CellMultiSelectProps = {
|
||||||
value: unknown;
|
value: unknown;
|
||||||
property: IBaseProperty;
|
property: IBaseProperty;
|
||||||
@@ -55,14 +62,55 @@ export function CellMultiSelect({
|
|||||||
[selectedIds, selectedSet, onCommit],
|
[selectedIds, selectedSet, onCommit],
|
||||||
);
|
);
|
||||||
|
|
||||||
|
const updatePropertyMutation = useUpdatePropertyMutation();
|
||||||
|
|
||||||
|
const trimmedSearch = search.trim();
|
||||||
|
const hasExactMatch = useMemo(
|
||||||
|
() =>
|
||||||
|
trimmedSearch.length > 0 &&
|
||||||
|
choices.some((c) => c.name.toLowerCase() === trimmedSearch.toLowerCase()),
|
||||||
|
[choices, trimmedSearch],
|
||||||
|
);
|
||||||
|
const showAddOption = trimmedSearch.length > 0 && !hasExactMatch;
|
||||||
|
|
||||||
|
const addOptionColor = useMemo(
|
||||||
|
() => CHOICE_COLORS[choices.length % CHOICE_COLORS.length],
|
||||||
|
[choices.length],
|
||||||
|
);
|
||||||
|
|
||||||
|
const handleAddOption = useCallback(() => {
|
||||||
|
if (!trimmedSearch) return;
|
||||||
|
const newChoice: Choice = {
|
||||||
|
id: uuid7(),
|
||||||
|
name: trimmedSearch,
|
||||||
|
color: addOptionColor,
|
||||||
|
};
|
||||||
|
const newChoices = [...choices, newChoice];
|
||||||
|
updatePropertyMutation.mutate({
|
||||||
|
propertyId: property.id,
|
||||||
|
baseId: property.baseId,
|
||||||
|
typeOptions: {
|
||||||
|
...typeOptions,
|
||||||
|
choices: newChoices,
|
||||||
|
choiceOrder: newChoices.map((c) => c.id),
|
||||||
|
},
|
||||||
|
});
|
||||||
|
onCommit([...selectedIds, newChoice.id]);
|
||||||
|
setSearch("");
|
||||||
|
}, [trimmedSearch, addOptionColor, choices, typeOptions, property, updatePropertyMutation, selectedIds, onCommit]);
|
||||||
|
|
||||||
const handleKeyDown = useCallback(
|
const handleKeyDown = useCallback(
|
||||||
(e: React.KeyboardEvent) => {
|
(e: React.KeyboardEvent) => {
|
||||||
if (e.key === "Escape") {
|
if (e.key === "Escape") {
|
||||||
e.preventDefault();
|
e.preventDefault();
|
||||||
onCancel();
|
onCancel();
|
||||||
}
|
}
|
||||||
|
if (e.key === "Enter" && showAddOption) {
|
||||||
|
e.preventDefault();
|
||||||
|
handleAddOption();
|
||||||
|
}
|
||||||
},
|
},
|
||||||
[onCancel],
|
[onCancel, showAddOption, handleAddOption],
|
||||||
);
|
);
|
||||||
|
|
||||||
const MAX_VISIBLE = 3;
|
const MAX_VISIBLE = 3;
|
||||||
@@ -110,6 +158,20 @@ export function CellMultiSelect({
|
|||||||
</span>
|
</span>
|
||||||
</div>
|
</div>
|
||||||
))}
|
))}
|
||||||
|
{showAddOption && (
|
||||||
|
<div
|
||||||
|
className={cellClasses.addOptionRow}
|
||||||
|
onClick={handleAddOption}
|
||||||
|
>
|
||||||
|
<span className={cellClasses.addOptionLabel}>Add option:</span>
|
||||||
|
<span
|
||||||
|
className={cellClasses.badge}
|
||||||
|
style={choiceColor(addOptionColor)}
|
||||||
|
>
|
||||||
|
{trimmedSearch}
|
||||||
|
</span>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
</div>
|
</div>
|
||||||
</Popover.Dropdown>
|
</Popover.Dropdown>
|
||||||
</Popover>
|
</Popover>
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
import { useState, useRef, useEffect, useCallback } from "react";
|
import { useState, useRef, useEffect, useCallback, useMemo } from "react";
|
||||||
import { Popover, TextInput } from "@mantine/core";
|
import { Popover, TextInput } from "@mantine/core";
|
||||||
import {
|
import {
|
||||||
IBaseProperty,
|
IBaseProperty,
|
||||||
@@ -6,8 +6,15 @@ import {
|
|||||||
Choice,
|
Choice,
|
||||||
} from "@/features/base/types/base.types";
|
} from "@/features/base/types/base.types";
|
||||||
import { choiceColor } from "@/features/base/components/cells/choice-color";
|
import { choiceColor } from "@/features/base/components/cells/choice-color";
|
||||||
|
import { useUpdatePropertyMutation } from "@/features/base/queries/base-property-query";
|
||||||
|
import { v7 as uuid7 } from "uuid";
|
||||||
import cellClasses from "@/features/base/styles/cells.module.css";
|
import cellClasses from "@/features/base/styles/cells.module.css";
|
||||||
|
|
||||||
|
const CHOICE_COLORS = [
|
||||||
|
"gray", "red", "pink", "grape", "violet", "indigo",
|
||||||
|
"blue", "cyan", "teal", "green", "lime", "yellow", "orange",
|
||||||
|
];
|
||||||
|
|
||||||
type CellSelectProps = {
|
type CellSelectProps = {
|
||||||
value: unknown;
|
value: unknown;
|
||||||
property: IBaseProperty;
|
property: IBaseProperty;
|
||||||
@@ -52,14 +59,54 @@ export function CellSelect({
|
|||||||
[selectedId, onCommit],
|
[selectedId, onCommit],
|
||||||
);
|
);
|
||||||
|
|
||||||
|
const updatePropertyMutation = useUpdatePropertyMutation();
|
||||||
|
|
||||||
|
const trimmedSearch = search.trim();
|
||||||
|
const hasExactMatch = useMemo(
|
||||||
|
() =>
|
||||||
|
trimmedSearch.length > 0 &&
|
||||||
|
choices.some((c) => c.name.toLowerCase() === trimmedSearch.toLowerCase()),
|
||||||
|
[choices, trimmedSearch],
|
||||||
|
);
|
||||||
|
const showAddOption = trimmedSearch.length > 0 && !hasExactMatch;
|
||||||
|
|
||||||
|
const addOptionColor = useMemo(
|
||||||
|
() => CHOICE_COLORS[choices.length % CHOICE_COLORS.length],
|
||||||
|
[choices.length],
|
||||||
|
);
|
||||||
|
|
||||||
|
const handleAddOption = useCallback(() => {
|
||||||
|
if (!trimmedSearch) return;
|
||||||
|
const newChoice: Choice = {
|
||||||
|
id: uuid7(),
|
||||||
|
name: trimmedSearch,
|
||||||
|
color: addOptionColor,
|
||||||
|
};
|
||||||
|
const newChoices = [...choices, newChoice];
|
||||||
|
updatePropertyMutation.mutate({
|
||||||
|
propertyId: property.id,
|
||||||
|
baseId: property.baseId,
|
||||||
|
typeOptions: {
|
||||||
|
...typeOptions,
|
||||||
|
choices: newChoices,
|
||||||
|
choiceOrder: newChoices.map((c) => c.id),
|
||||||
|
},
|
||||||
|
});
|
||||||
|
onCommit(newChoice.id);
|
||||||
|
}, [trimmedSearch, addOptionColor, choices, typeOptions, property, updatePropertyMutation, onCommit]);
|
||||||
|
|
||||||
const handleKeyDown = useCallback(
|
const handleKeyDown = useCallback(
|
||||||
(e: React.KeyboardEvent) => {
|
(e: React.KeyboardEvent) => {
|
||||||
if (e.key === "Escape") {
|
if (e.key === "Escape") {
|
||||||
e.preventDefault();
|
e.preventDefault();
|
||||||
onCancel();
|
onCancel();
|
||||||
}
|
}
|
||||||
|
if (e.key === "Enter" && showAddOption) {
|
||||||
|
e.preventDefault();
|
||||||
|
handleAddOption();
|
||||||
|
}
|
||||||
},
|
},
|
||||||
[onCancel],
|
[onCancel, showAddOption, handleAddOption],
|
||||||
);
|
);
|
||||||
|
|
||||||
if (isEditing) {
|
if (isEditing) {
|
||||||
@@ -112,6 +159,20 @@ export function CellSelect({
|
|||||||
</span>
|
</span>
|
||||||
</div>
|
</div>
|
||||||
))}
|
))}
|
||||||
|
{showAddOption && (
|
||||||
|
<div
|
||||||
|
className={cellClasses.addOptionRow}
|
||||||
|
onClick={handleAddOption}
|
||||||
|
>
|
||||||
|
<span className={cellClasses.addOptionLabel}>Add option:</span>
|
||||||
|
<span
|
||||||
|
className={cellClasses.badge}
|
||||||
|
style={choiceColor(addOptionColor)}
|
||||||
|
>
|
||||||
|
{trimmedSearch}
|
||||||
|
</span>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
</div>
|
</div>
|
||||||
</Popover.Dropdown>
|
</Popover.Dropdown>
|
||||||
</Popover>
|
</Popover>
|
||||||
|
|||||||
@@ -167,6 +167,7 @@ function NumberOptions({
|
|||||||
<Select
|
<Select
|
||||||
size="xs"
|
size="xs"
|
||||||
label={t("Format")}
|
label={t("Format")}
|
||||||
|
allowDeselect={false}
|
||||||
data={[
|
data={[
|
||||||
{ value: "plain", label: t("Number") },
|
{ value: "plain", label: t("Number") },
|
||||||
{ value: "currency", label: t("Currency") },
|
{ value: "currency", label: t("Currency") },
|
||||||
@@ -175,7 +176,7 @@ function NumberOptions({
|
|||||||
]}
|
]}
|
||||||
value={options?.format ?? "plain"}
|
value={options?.format ?? "plain"}
|
||||||
onChange={(val) =>
|
onChange={(val) =>
|
||||||
onUpdate({ ...property.typeOptions, format: val })
|
onUpdate({ ...property.typeOptions, format: val ?? "plain" })
|
||||||
}
|
}
|
||||||
/>
|
/>
|
||||||
<NumberInput
|
<NumberInput
|
||||||
@@ -219,13 +220,14 @@ function DateOptions({
|
|||||||
<Select
|
<Select
|
||||||
size="xs"
|
size="xs"
|
||||||
label={t("Time format")}
|
label={t("Time format")}
|
||||||
|
allowDeselect={false}
|
||||||
data={[
|
data={[
|
||||||
{ value: "12h", label: "12-hour" },
|
{ value: "12h", label: "12-hour" },
|
||||||
{ value: "24h", label: "24-hour" },
|
{ value: "24h", label: "24-hour" },
|
||||||
]}
|
]}
|
||||||
value={options?.timeFormat ?? "12h"}
|
value={options?.timeFormat ?? "12h"}
|
||||||
onChange={(val) =>
|
onChange={(val) =>
|
||||||
onUpdate({ ...property.typeOptions, timeFormat: val })
|
onUpdate({ ...property.typeOptions, timeFormat: val ?? "12h" })
|
||||||
}
|
}
|
||||||
/>
|
/>
|
||||||
)}
|
)}
|
||||||
|
|||||||
@@ -264,3 +264,26 @@
|
|||||||
.menuItem:hover {
|
.menuItem:hover {
|
||||||
background-color: light-dark(var(--mantine-color-gray-0), var(--mantine-color-dark-5));
|
background-color: light-dark(var(--mantine-color-gray-0), var(--mantine-color-dark-5));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.addOptionRow {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
gap: 8px;
|
||||||
|
padding: 6px 8px;
|
||||||
|
cursor: pointer;
|
||||||
|
border-radius: var(--mantine-radius-sm);
|
||||||
|
transition: background-color 100ms ease;
|
||||||
|
border-top: 1px solid light-dark(var(--mantine-color-gray-2), var(--mantine-color-dark-4));
|
||||||
|
margin-top: 4px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.addOptionRow:hover {
|
||||||
|
background-color: light-dark(var(--mantine-color-gray-0), var(--mantine-color-dark-5));
|
||||||
|
}
|
||||||
|
|
||||||
|
.addOptionLabel {
|
||||||
|
font-size: var(--mantine-font-size-xs);
|
||||||
|
color: light-dark(var(--mantine-color-gray-5), var(--mantine-color-dark-3));
|
||||||
|
white-space: nowrap;
|
||||||
|
flex-shrink: 0;
|
||||||
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user