feat: get rid of context and made useModal can be execute everywhere

Author: hunterliu1003

docs/content/2.get-started/1.guide/4.types.md

@@ -33,7 +33,6 @@ export type ModalSlot = string | Component | ModalSlotOptions
 export type UseModalOptions<P> = {
   defaultModelValue?: boolean
   keepAlive?: boolean
-  context?: Vfm
   component?: Constructor<P>
   attrs?: (RawProps & P) | ({} extends P ? null : never)
   slots?: {

packages/vue-final-modal/cypress/components/focusTrap.spec.ts

@@ -7,7 +7,6 @@ describe('Test focusTrap', () => {
     const vfm = createVfm()
 
     const firstModal = useModal({
-      context: vfm,
       component: VueFinalModal,
       attrs: { contentClass: 'first-modal-content' },
       slots: {
@@ -16,7 +15,6 @@ describe('Test focusTrap', () => {
     })
 
     const secondModal = useModal({
-      context: vfm,
       component: VueFinalModal,
       attrs: { contentClass: 'second-modal-content' },
       slots: {

packages/vue-final-modal/cypress/components/useModal.spec.ts

@@ -6,7 +6,6 @@ describe('Test useModal()', () => {
   it('Should be closed by default', () => {
     const vfm = createVfm()
     const modal = useModal({
-      context: vfm,
       slots: { default: 'Hello World!' },
     })
 
@@ -25,7 +24,6 @@ describe('Test useModal()', () => {
   it('Should be opened by given defaultModelValue: true', () => {
     const vfm = createVfm()
     useModal({
-      context: vfm,
       defaultModelValue: true,
       slots: {
         default: 'Hello World!',
@@ -51,7 +49,6 @@ describe('Test useModal()', () => {
     const onClosed = cy.spy().as('onClosed')
 
     const modal = useModal({
-      context: vfm,
       attrs: {
         onBeforeOpen,
         onOpened,

packages/vue-final-modal/cypress/components/useZIndex.spec.ts

@@ -5,19 +5,16 @@ describe('Test useZIndex()', () => {
   it('Props: zIndexFn()', () => {
     const vfm = createVfm()
     const firstModal = useModal({
-      context: vfm,
       component: VueFinalModal,
       attrs: { class: 'first-modal' },
     })
 
     const secondModal = useModal({
-      context: vfm,
       component: VueFinalModal,
       attrs: { class: 'second-modal' },
     })
 
     const thirdModal = useModal({
-      context: vfm,
       component: VueFinalModal,
       attrs: { class: 'third-modal' },
     })

packages/vue-final-modal/src/Modal.ts

@@ -25,7 +25,6 @@ export type ModalSlot = string | Component | ModalSlotOptions
 export type UseModalOptions<P> = {
   defaultModelValue?: boolean
   keepAlive?: boolean
-  context?: Vfm
   component?: Constructor<P>
   attrs?: (RawProps & P) | ({} extends P ? null : never)
   slots?: {

packages/vue-final-modal/src/useApi.ts

@@ -1,19 +1,23 @@
+import { computed, inject, markRaw, nextTick, reactive, useAttrs } from 'vue'
 import { isString, tryOnUnmounted } from '@vueuse/core'
-import { computed, getCurrentInstance, inject, markRaw, reactive, useAttrs } from 'vue'
 import type { Component } from 'vue'
 import VueFinalModal from './components/VueFinalModal/VueFinalModal.vue'
 import type CoreModal from './components/CoreModal/CoreModal.vue'
-import { internalVfmSymbol, vfmSymbol } from './injectionSymbols'
+import { internalVfmSymbol } from './injectionSymbols'
 
 import type { ComponentProps, Constructor, InternalVfm, ModalSlot, ModalSlotOptions, RawProps, UseModalOptions, UseModalOptionsPrivate, UseModalReturnType, Vfm } from './Modal'
-import { activeVfm, getActiveVfm, setActiveVfm } from './plugin'
+import { activeVfm, getActiveVfm } from './plugin'
 
 /**
  * Returns the vfm instance. Equivalent to using `$vfm` inside
  * templates.
  */
 export function useVfm(): Vfm {
-  return getActiveVfm()!
+  const vfm = getActiveVfm()
+  if (__DEV__ && !vfm)
+    consoleError()
+
+  return vfm!
 }
 
 /**
@@ -53,25 +57,8 @@ function withMarkRaw<P>(options: Partial<UseModalOptions<P>>, DefaultComponent:
  * Create a dynamic modal.
  */
 export function useModal<P = InstanceType<typeof VueFinalModal>['$props']>(_options: UseModalOptions<P>): UseModalReturnType<P> {
-  const currentInstance = getCurrentInstance()
-  let vfm = _options.context || (currentInstance && inject(vfmSymbol))
-  if (vfm)
-    setActiveVfm(vfm)
-
-  if (__DEV__ && !activeVfm) {
-    throw new Error(
-      '[🍍]: getActiveVfm was called with no active Vfm. Did you forget to install vfm?\n'
-        + '\tconst vfm = createVfm()\n'
-        + '\tapp.use(vfm)\n'
-        + 'This will fail in production.',
-    )
-  }
-
-  vfm = activeVfm
-
   const options = reactive({
     id: Symbol('useModal'),
-    context: vfm,
     modelValue: !!_options?.defaultModelValue,
     resolveOpened: () => { },
     resolveClosed: () => { },
@@ -83,16 +70,26 @@ export function useModal<P = InstanceType<typeof VueFinalModal>['$props']>(_opti
       destroy()
   })
 
-  if (options.modelValue === true)
-    options.context?.dynamicModals.push(options)
+  if (options.modelValue === true) {
+    nextTick(() => {
+      const vfm = useVfm()
+      vfm?.dynamicModals.push(options)
+    })
+  }
 
-  function open(): Promise<string> {
+  async function open(): Promise<string> {
+    await nextTick()
+    const vfm = useVfm()
+    if (!vfm) {
+      consoleError()
+      return Promise.resolve('error')
+    }
     if (options.modelValue)
       return Promise.resolve('[Vue Final Modal] modal is already opened.')
 
     destroy()
     options.modelValue = true
-    options.context?.dynamicModals.push(options)
+    vfm.dynamicModals.push(options)
 
     return new Promise((resolve) => {
       options.resolveOpened = () => resolve('opened')
@@ -116,8 +113,6 @@ export function useModal<P = InstanceType<typeof VueFinalModal>['$props']>(_opti
       options.defaultModelValue = _options.defaultModelValue
     if (_options?.keepAlive !== undefined)
       options.keepAlive = _options?.keepAlive
-    if (_options.context)
-      options.context = _options.context
 
     // patch options.component and options.attrs
     patchComponentOptions(options, rest)
@@ -156,11 +151,14 @@ export function useModal<P = InstanceType<typeof VueFinalModal>['$props']>(_opti
   }
 
   function destroy(): void {
-    if (!options.context)
+    const vfm = useVfm()
+    if (!vfm) {
+      consoleError()
       return
-    const index = options.context.dynamicModals.indexOf(options)
+    }
+    const index = vfm.dynamicModals.indexOf(options)
     if (index !== -1)
-      options.context.dynamicModals.splice(index, 1)
+      vfm.dynamicModals.splice(index, 1)
   }
 
   return {
@@ -223,3 +221,14 @@ export function useVfmAttrs(options: {
 
   return vfmAttrs
 }
+
+function consoleError() {
+  if (__DEV__ && !activeVfm) {
+    throw new Error(
+      '[Vue Final Modal]: getActiveVfm was called with no active Vfm. Did you forget to install vfm?\n'
+        + '\tconst vfm = createVfm()\n'
+        + '\tapp.use(vfm)\n'
+        + 'This will fail in production.',
+    )
+  }
+}

viteplay/src/components/VueFinalModal/Basic.example.vue

@@ -2,10 +2,13 @@
 import { ref } from 'vue'
 import { ModalsContainer, VueFinalModal, useModal, useModalSlot, useVfm } from 'vue-final-modal'
 import DefaultSlot from '../DefaultSlot.vue'
+import { modal } from './modalsHelpers'
 import TestModal from './TestModal.vue'
 
-const { toggle, closeAll } = useVfm()
+console.log('modal → ', modal)
 
+const { toggle, closeAll } = useVfm()
+// modal.open()
 const modal1 = useModal({
   keepAlive: true,
   component: VueFinalModal,

viteplay/src/components/VueFinalModal/modalsHelpers.ts

@@ -0,0 +1,19 @@
+import { VueFinalModal, useModal, useModalSlot } from 'vue-final-modal'
+import DefaultSlot from '../DefaultSlot.vue'
+
+export const modal = useModal({
+  component: VueFinalModal,
+  // defaultModelValue: true,
+  slots: {
+    default: useModalSlot({
+      component: DefaultSlot,
+      attrs: {
+        text: '123',
+        onCreate() {
+          // console.log('onCreated')
+        },
+      },
+    }),
+  },
+})
+// modal.open()