Skip to content

redundant solved#79

Merged
NimaMaria merged 3 commits into
NimaMaria:mainfrom
annrose2277-glitch:supreme
Mar 13, 2026
Merged

redundant solved#79
NimaMaria merged 3 commits into
NimaMaria:mainfrom
annrose2277-glitch:supreme

Conversation

@annrose2277-glitch
Copy link
Copy Markdown
Contributor

📌 Description

I have fixed the Redundant Backdrop & Dialog Behavior bug by removing the manual backdrop div and ensuring the native behavior
is used.

🔗 Related Issue

Closes #63


🛠 Changes Made

HTML (index.html)
Removed the manual

element. This div was redundant because the native element provides its own backdrop.

Removed

CSS (style.css)
Verified that styling is correctly applied to the native backdrop using the ::backdrop pseudo-element. This ensures the background still
has the desired aesthetic (blur and tint) without needing a separate DOM element.

modal::backdrop{
background: rgba(255,215,230,0.4);
backdrop-filter: blur(5px);
}

JavaScript (app.js)
Verified that the logic for opening and closing the modal correctly uses the native showModal() and close() methods. Since the manual
backdrop was already absent from the JS logic, no changes were needed here, ensuring a clean and direct implementation.

📷 Screenshots (if applicable)


✅ Checklist

  • [yes ] I have tested my changes
  • [ yes] My code follows project guidelines
  • [ yes] I have linked the related issue

@github-actions github-actions Bot added bug Something isn't working HackTheStack level-1 labels Mar 12, 2026
@NimaMaria
Copy link
Copy Markdown
Owner

This branch has merge conflict

@NimaMaria NimaMaria merged commit 39683f0 into NimaMaria:main Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working HackTheStack level-1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redundant Backdrop & Dialog Behavior

2 participants