Skip to content

Commit

Permalink
fix: do not need to execute refreshZIndex in CoreModal
Browse files Browse the repository at this point in the history
  • Loading branch information
hunterliu1003 committed Feb 26, 2023
1 parent 675225b commit 1cac1a2
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 3 deletions.
Binary file modified packages/vue-final-modal/cypress/videos/focusTrap.spec.ts.mp4
Binary file not shown.
Binary file modified packages/vue-final-modal/cypress/videos/useModal.spec.ts.mp4
Binary file not shown.
Binary file modified packages/vue-final-modal/cypress/videos/useZIndex.spec.ts.mp4
Binary file not shown.
Expand Up @@ -114,7 +114,7 @@ const modalInstance = computed<Modal>(() => ({
},
}))
const { zIndex, refreshZIndex } = useZIndex(props, { openedModals, modalInstance, visible })
const { zIndex } = useZIndex(props, { openedModals, modalInstance, visible })
onMounted(() => {
modals.push(modalInstance)
Expand All @@ -130,7 +130,6 @@ watch(modelValueLocal, (value) => {
async function open() {
emitEvent('beforeOpen')
moveToLastOpenedModals(modalInstance)
refreshZIndex()
openLastOverlay()
enterTransition()
}
Expand Down
Expand Up @@ -28,6 +28,5 @@ export function useZIndex(

return {
zIndex,
refreshZIndex,
}
}

0 comments on commit 1cac1a2

Please sign in to comment.