CodeMirror/budibase
1
0
Fork 0
mirror of synced 2024-08-10 23:51:24 +12:00
Code Issues Projects Releases Wiki Activity

Replace incorrect usages of element.childNodes with element.children

Browse source
This commit is contained in:
Andrew Kingston 2022-07-15 13:47:39 +01:00
parent 6ee5a78b0c
commit c9ee465507
5 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/builder/src/pages/builder/app/[application]/design/[screenId]/components/[componentId]/_components/navigation/DNDPositionIndicator.svelte
View file

@ -16,7 +16,7 @@
// Get root li element
const el = document.getElementById(`component-${component?._id}`)
// Get inner nav item content element
const child = el?.childNodes[0]?.childNodes[0]
const child = el?.children[0]?.children[0]
if (!el) {
return
}

2
packages/client/src/components/Component.svelte
View file

@ -399,7 +399,7 @@
}
const scrollIntoView = () => {
const node = document.getElementsByClassName(id)?.[0]?.childNodes[0]
const node = document.getElementsByClassName(id)?.[0]?.children[0]
if (!node) {
return
}

4
packages/client/src/components/preview/DNDHandler.svelte
View file

@ -35,8 +35,8 @@
const getDOMNodeForComponent = component => {
const parent = component.closest(".component")
const children = Array.from(parent.childNodes)
return children?.find(node => node?.nodeType === 1)
const children = Array.from(parent.children)
return children[0]
}
// Callback when initially starting a drag on a draggable component

3
packages/client/src/components/preview/IndicatorSet.svelte
View file

@ -71,8 +71,7 @@
// Extract valid children
// Sanity limit of 100 active indicators
const children = Array.from(parents)
.map(parent => parent?.childNodes?.[0])
.filter(node => node?.nodeType === 1)
.map(parent => parent?.children?.[0])
.slice(0, 100)
// If there aren't any nodes then reset

2
packages/client/src/components/preview/SettingsBar.svelte
View file

@ -37,7 +37,7 @@
}
const id = $builderStore.selectedComponentId
const parent = document.getElementsByClassName(id)?.[0]
const element = parent?.childNodes?.[0]
const element = parent?.children?.[0]
// The settings bar is higher in the dom tree than the selection indicators
// as we want to be able to render the settings bar wider than the screen,