@koumoul/vjsf 3.20.6 → 3.20.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@koumoul/vjsf",
-  "version": "3.20.6",
+  "version": "3.20.8",
   "description": "Generate forms for the vuetify UI library (vuejs) based on annotated JSON schemas.",
   "scripts": {
     "test-tz1": "TZ=Europe/Paris vitest run",

package/src/components/fragments/help-message.vue

@@ -5,6 +5,7 @@
       :class="`vjsf-help-message-tooltip vjsf-help-message-tooltip-${node.options.density}`"
       location="top end"
       offset="4"
+      :z-index="zIndex"
       :close-on-content-click="false"
     >
       <template #activator="{ props }">
@@ -42,8 +43,9 @@ import { VAlert } from 'vuetify/components/VAlert'
 import { VMenu } from 'vuetify/components/VMenu'
 import { VBtn } from 'vuetify/components/VBtn'
 import { ref } from 'vue'
+import useZIndexStack from '../../composables/use-z-index-stack.js'
 
-defineProps({
+const props = defineProps({
   node: {
     /** @type import('vue').PropType<import('../../types.js').VjsfNode> */
     type: Object,
@@ -51,6 +53,8 @@ defineProps({
   }
 })
 
+const zIndex = useZIndexStack(props.node.fullKey)
+
 const show = ref(false)
 </script>
 

package/src/components/fragments/text-field-menu.vue

@@ -55,14 +55,14 @@ const fieldProps = computed(() => {
   return fieldProps
 })
 
-const zIndex = useZIndexStack(() => props.modelValue.fullKey)
+const zIndex = useZIndexStack(props.modelValue.fullKey)
 
 const menuProps = computed(() => {
   return {
     ...compProps.value,
     minWidth: props.minWidth,
     maxWidth: props.maxWidth,
-    zIndex: zIndex.value,
+    zIndex,
     closeOnContentClick: false,
     disabled: true
   }

package/src/components/nodes/combobox.vue

@@ -25,7 +25,7 @@ export default defineComponent({
     const nodeRef = toRef(props, 'modelValue')
     const getItems = useGetItems(nodeRef, props.statefulLayout)
     const { inputProps, compSlots, localData, layout, options } = useNode(nodeRef, props.statefulLayout)
-    const zIndex = useZIndexStack(() => props.modelValue.fullKey)
+    const zIndex = useZIndexStack(props.modelValue.fullKey)
 
     const fieldProps = computed(() => {
       const fieldProps = { ...inputProps.value }
@@ -33,7 +33,7 @@ export default defineComponent({
       if (options.value.readOnly) {
         fieldProps.menuProps = { modelValue: false }
       } else {
-        fieldProps.menuProps = { zIndex: zIndex.value }
+        fieldProps.menuProps = { zIndex }
       }
       if (getItems.hasItems.value) {
         fieldProps.items = getItems.items.value

package/src/components/nodes/list.vue

@@ -25,6 +25,7 @@ import useZIndexStack from '../../composables/use-z-index-stack.js'
 useDefaults({}, 'VjsfList')
 const vCardProps = useCompDefaults('VjsfList-VCard', { border: true, flat: true, tile: true })
 const vEditDialogProps = useCompDefaults('VjsfList-Edit-VDialog', { width: 500, persistent: true })
+const vEditDialogVSheetProps = useCompDefaults('VjsfList-Edit-VDialog-VSheet', {})
 const vEditMenuProps = useCompDefaults('VjsfList-Edit-VMenu', { width: 500 })
 const theme = useTheme()
 
@@ -69,7 +70,7 @@ const prepareDrag = (/** @type {number} */index) => {
   menuOpened.value = -1
 }
 
-const zIndex = useZIndexStack(() => props.modelValue.fullKey)
+const zIndex = useZIndexStack(props.modelValue.fullKey)
 
 /* manage hovered and edited items */
 // const editedItem = computed(() => activatedItems.value[fullKey.value])
@@ -494,8 +495,9 @@ const itemBorderColor = computed(() => (/** @type {import('@json-layout/core').S
         v-if="layout.listEditMode === 'dialog'"
         :model-value="editedItem !== undefined"
         v-bind="vEditDialogProps"
+        :z-index="zIndex"
       >
-        <v-sheet>
+        <v-sheet v-bind="vEditDialogVSheetProps">
           <v-toolbar
             density="compact"
             color="surface"

package/src/components/nodes/number-combobox.vue

@@ -25,7 +25,7 @@ export default defineComponent({
     const nodeRef = toRef(props, 'modelValue')
     const getItems = useGetItems(nodeRef, props.statefulLayout)
     const { inputProps, compSlots, localData, layout, options } = useNode(nodeRef, props.statefulLayout, { bindData: false, layoutPropsMap: ['step', 'min', 'max'] })
-    const zIndex = useZIndexStack(() => props.modelValue.fullKey)
+    const zIndex = useZIndexStack(props.modelValue.fullKey)
 
     const fieldProps = computed(() => {
       const fieldProps = { ...inputProps.value }
@@ -34,7 +34,7 @@ export default defineComponent({
       if (options.value.readOnly) {
         fieldProps.menuProps = { modelValue: false }
       } else {
-        fieldProps.menuProps = { zIndex: zIndex.value }
+        fieldProps.menuProps = { zIndex }
       }
       if (getItems.hasItems.value) {
         fieldProps.items = getItems.items.value

package/src/components/nodes/one-of-select.vue

@@ -27,7 +27,7 @@ const props = defineProps({
 const { inputProps, localData, skeleton, children } = useNode(
   toRef(props, 'modelValue'), props.statefulLayout, { bindData: false }
 )
-const zIndex = useZIndexStack(() => props.modelValue.fullKey)
+const zIndex = useZIndexStack(props.modelValue.fullKey)
 
 /** @type import('vue').Ref<string | undefined> */
 const activeChildTree = ref(undefined)
@@ -50,7 +50,7 @@ const fieldProps = computed(() => {
   const fieldProps = { ...inputProps.value }
   fieldProps['onUpdate:modelValue'] = onChange
   if (!props.modelValue.options.readOnly) {
-    fieldProps.menuProps = { zIndex: zIndex.value }
+    fieldProps.menuProps = { zIndex }
   }
   const items = []
   for (const childTreePointer of skeleton.value.childrenTrees || []) {

package/src/composables/use-get-items.js

@@ -18,8 +18,11 @@ export default function (nodeRef, statefulLayout) {
 
   const fetchItems = async () => {
     loading.value = true
-    items.value = await statefulLayout.getItems(nodeRef.value, search.value)
-    loading.value = false
+    try {
+      items.value = await statefulLayout.getItems(nodeRef.value, search.value)
+    } finally {
+      loading.value = false
+    }
   }
 
   watch(() => nodeRef.value.itemsCacheKey, (newValue, oldValue) => {

package/src/composables/use-select-node.js

@@ -17,7 +17,7 @@ export default function (nodeRef, statefulLayout, avatarProps, cssPrefix) {
 
   const { inputProps, options, skeleton, localData, compSlots } = useField(nodeRef, statefulLayout, { layoutPropsMap: ['multiple', 'placeholder'], bindData: false })
   const getItems = useGetItems(nodeRef, statefulLayout)
-  const zIndex = useZIndexStack(() => nodeRef.value.fullKey)
+  const zIndex = useZIndexStack(nodeRef.value.fullKey)
 
   const selectProps = computed(() => {
     const props = { ...inputProps.value }
@@ -25,7 +25,7 @@ export default function (nodeRef, statefulLayout, avatarProps, cssPrefix) {
     if (options.value.readOnly) {
       props.menuProps = { modelValue: false }
     } else {
-      props.menuProps = { zIndex: zIndex.value }
+      props.menuProps = { zIndex }
     }
     props.clearable = props.clearable ?? !skeleton.value.required
     props.valueComparator = (/** @type {any} */a, /** @type {any} */b) => {

package/src/composables/use-z-index-stack.js

@@ -1,21 +1,18 @@
 // vuetify zIndex stacking is buggy (for example https://github.com/vuetifyjs/vuetify/issues/16251)
 // we try to work around this in this composable
 
-import { provide, inject, computed } from 'vue'
+import { provide, inject } from 'vue'
 
 const symbol = Symbol.for('vjsf:z-index-stack')
 
 /**
- * @param {() => string} keyGetter
+ * @param {string} fullKey
  */
-export default (keyGetter) => {
-  return computed(() => {
-    const fullKey = keyGetter()
-    const currentZIndex = inject(symbol, { zIndex: 3000, fullKey })
-    const newZIndex = currentZIndex.zIndex + 10
-    if (fullKey.length > currentZIndex.fullKey.length) {
-      provide(symbol, { zIndex: newZIndex, fullKey })
-    }
-    return newZIndex
-  })
+export default (fullKey) => {
+  const currentZIndex = inject(symbol, { zIndex: 3000, fullKey, initial: true })
+  const newZIndex = currentZIndex.zIndex + 10
+  if (currentZIndex.initial || fullKey.length > currentZIndex.fullKey.length) {
+    provide(symbol, { zIndex: newZIndex, fullKey })
+  }
+  return newZIndex
 }

package/src/styles/vjsf.css

@@ -21,8 +21,8 @@
   display: none;
 }
 .vjsf-node.vjsf-has-help {
-  padding-right: 28px!important;
+  padding-right: 30px!important;
 }
 .vjsf-node.vjsf-density-default.vjsf-has-help {
-  padding-right: 32px!important;
+  padding-right: 42px!important;
 }

package/types/composables/use-get-items.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-get-items.d.ts","sourceRoot":"","sources":["../../src/composables/use-get-items.js"],"names":[],"mappings":"AAEA;;;GAGG;AACH,0CAHW,OAAO,KAAK,EAAE,GAAG,CAAC,OAAO,aAAa,EAAE,QAAQ,CAAC,kBACjD,OAAO,aAAa,EAAE,kBAAkB;;;;;wCA8BtC,GAAG,aACH,GAAG;EAiBf"}
+{"version":3,"file":"use-get-items.d.ts","sourceRoot":"","sources":["../../src/composables/use-get-items.js"],"names":[],"mappings":"AAEA;;;GAGG;AACH,0CAHW,OAAO,KAAK,EAAE,GAAG,CAAC,OAAO,aAAa,EAAE,QAAQ,CAAC,kBACjD,OAAO,aAAa,EAAE,kBAAkB;;;;;wCAiCtC,GAAG,aACH,GAAG;EAiBf"}

package/types/composables/use-z-index-stack.d.ts

@@ -1,3 +1,3 @@
-declare function _default(keyGetter: () => string): import("vue").ComputedRef<number>;
+declare function _default(fullKey: string): number;
 export default _default;
 //# sourceMappingURL=use-z-index-stack.d.ts.map

package/types/composables/use-z-index-stack.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-z-index-stack.d.ts","sourceRoot":"","sources":["../../src/composables/use-z-index-stack.js"],"names":[],"mappings":"AAUe,qCAFJ,MAAM,MAAM,qCAYtB"}
+{"version":3,"file":"use-z-index-stack.d.ts","sourceRoot":"","sources":["../../src/composables/use-z-index-stack.js"],"names":[],"mappings":"AAUe,mCAFJ,MAAM,UAShB"}