Skip to content

Conversation

JasperX777
Copy link

@JasperX777 JasperX777 commented Aug 9, 2025

JasperX777 Quick JasperX777 /jasper/fix-1886-modal-position → Lissy93/dashy Commits: 1 | Files Changed: 1 | Additions: 0 Label Unchecked Tasks Powered by Pull Request Badge

Category:

Bugfix

Overview

Fixes an issue where modals in the "glass" and "glass-2" themes were not centered on the screen and appeared “trapped” inside the edit mode area.
The root cause was "div.edit-mode-bottom-banner" having "backdrop-filter: blur(50px)", which created an unexpected stacking context and interfered with modal positioning.
By removing this backdrop filter, the modals are now positioned correctly in the center of the viewport.

Issue Number (if applicable) #1886 Configuration nenue stuck at line

New Vars (if applicable)

Screenshot (if applicable)
Before
截屏2025-08-09 下午3 20 48

After
截屏2025-08-09 下午3 17 07

Code Quality Checklist (Please complete)

  • All changes are backwards compatible
  • All lint checks and tests are passing
  • There are no (new) build warnings or errors
  • (If a new config option is added) Attribute is outlined in the schema and documented
  • (If a new dependency is added) Package is essential, and has been checked out for security or performance
  • (If significant change) Bumps version in package.json

@JasperX777 JasperX777 requested a review from Lissy93 as a code owner August 9, 2025 03:22
Copy link

netlify bot commented Aug 9, 2025

Deploy Preview for dashy-dev ready!

Name Link
🔨 Latest commit 0d1d49a
🔍 Latest deploy log https://app.netlify.com/projects/dashy-dev/deploys/6896bee446aa0e0009b8e62f
😎 Deploy Preview https://deploy-preview-1899--dashy-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant