fix: set `MODAL_STACK_Z_INDEX` to `10000`

[xhuajin]

Description

When using the desktop version (Windows 10), I noticed that when the ‘Trending’ section is opened, the ‘Minimize, Reduce, Close’ buttons in the top right corner overlap the ‘Trending’ modal, obscuring about a third of the ‘Trending’ close button.

bug screenshot:

Upon inspection, it was found that the z-index of the three buttons in the top right corner was set to 9999, whereas the z-index of the ‘Trending’ modal was only 1001. After attempting to change the z-index of the three buttons from 9999 to 1000, everything appeared to function normally. However, to minimize the impact, this PR opts to resolve the issue by increasing the z-index of the modal from 1001 to 10000.

Screenshot after repair:

[vercel[bot]]

@xhuajin is attempting to deploy a commit to the RSS3 Team on Vercel.

A member of the Team first needs to authorize it.

[lawvs]

Simply increasing the z-index value isn't the best solution. It's generally expected for the application's action buttons to remain on the top layer.

[Innei]

This implementation doesn't make sense, you should adjust the boundaries of the modal instead of the Z-index.

[xhuajin]

According to your suggestion, I remove the previous method of modifying z-index and add 'top: ${ElECTRON_CUSTOM_TITLEBAR_HEIGHT}px' to solve this problem.

Now the trending modal will look like:

[Innei]

Sorry, I just fixed this, there is a built-in css atom class named safe-inset-top-4, you can use it.

[xhuajin]

Got it. Thanks for your reply