fix: Export popup has small z-index [INS-3640]

[CurryYangxx]

Close #7176

Changes

• remove the global variable(globalZIndex) in base modal component
• add a new hook to admin every modal z-index(Even without using the base modal component)
• change the export request modal z-index, use globalModalZIndex instead

How to test

• Click Preferences → Data -> Export the collection, and you can see the export modal

[ihexxa]

Thanks for the quick fix, good job👍.

[jackkav]

Rather than syncing the zlndex's with a hook we could try to remove the zindex settings? @gatzjames ?

[gatzjames]

We are moving away from the base/modal and using react-aria-components modals instead.
I'd suggest changing the settings-modal.tsx to have the same z-index as other modals instead.

In settings-modal.tsx line 44

<Modal className='!z-10' ref={modalRef} tall {...props}>
   ...
</Modal>

That way the last modal to open will always be on top and there's no need to keep calculations in state.

[CurryYangxx]

We are moving away from the base/modal and using react-aria-components modals instead. I'd suggest changing the settings-modal.tsx to have the same z-index as other modals instead.

In settings-modal.tsx line 44

<Modal className='!z-10' ref={modalRef} tall {...props}>
   ...
</Modal>

That way the last modal to open will always be on top and there's no need to keep calculations in state.

This way works. And do we have other similar scenes that import the base/modal? I'm worried that if there are similar scenes that are not covered by the same z-index, there may be the same issue. @gatzjames

[gatzjames]

This way works. And do we have other similar scenes that import the base/modal? I'm worried that if there are similar scenes that are not covered by the same z-index, there may be the same issue. @gatzjames

There are definitely more modals that use the previous approach.
I don't think there are other cases where we have this issue especially since opening a modal on top of another modal is offering a bad UX. So our goal is to eliminate these issues by migrating these modals instead of investing more on the previous approach