CodeMirror/budibase
1
0
Fork 0
mirror of synced 2024-09-12 23:43:09 +12:00
Code Issues Projects Releases Wiki Activity

Fixing an issue with ID/Rev not being copyable from context menu due to the way IDs were split/handled in the grid.

Browse source
This commit is contained in:
mike12345567 2024-05-09 16:18:43 +01:00
parent 0c3bc3e13e
commit 60f8ce26af
1 changed files with 12 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
packages/frontend-core/src/components/grid/stores/ui.js
View file

@ -8,6 +8,15 @@ import {
NewRowID,
} from "../lib/constants"
function splitRowId(rowId) {
if (!rowId) {
return undefined
}
const parts = rowId.split("-")
const field = parts.pop()
return { id: parts.join("-"), field }
}
export const createStores = context => {
const { props } = context
const focusedCellId = writable(null)
@ -25,7 +34,7 @@ export const createStores = context => {
const focusedRowId = derived(
focusedCellId,
$focusedCellId => {
return $focusedCellId?.split("-")[0]
return splitRowId($focusedCellId)?.id
},
null
)
@ -72,7 +81,7 @@ export const deriveStores = context => {
const focusedRow = derived(
[focusedCellId, rowLookupMap, rows],
([$focusedCellId, $rowLookupMap, $rows]) => {
const rowId = $focusedCellId?.split("-")[0]
const rowId = splitRowId($focusedCellId)?.id
// Edge case for new rows
if (rowId === NewRowID) {
@ -152,7 +161,7 @@ export const initialise = context => {
const hasRow = rows.actions.hasRow
// Check selected cell
const selectedRowId = $focusedCellId?.split("-")[0]
const selectedRowId = splitRowId($focusedCellId)?.id
if (selectedRowId && !hasRow(selectedRowId)) {
focusedCellId.set(null)
}