Skip to content

fix: Remove malformed header markup and duplicate debugger toggle button#96

Merged
siddu-k merged 2 commits into
siddu-k:mainfrom
akash3911:fix/duplicate-debugger-toggle
May 28, 2026
Merged

fix: Remove malformed header markup and duplicate debugger toggle button#96
siddu-k merged 2 commits into
siddu-k:mainfrom
akash3911:fix/duplicate-debugger-toggle

Conversation

@akash3911

Copy link
Copy Markdown
Contributor

Summary

This PR fixes malformed toolbar markup in index.html caused by:

  • Duplicate id="btn-debugger-toggle" elements
  • An unclosed <button> tag
  • Nested toolbar buttons inside malformed markup

Changes Made

  • Removed the incomplete debugger toggle button
  • Kept only a single valid btn-debugger-toggle element
  • Properly closed toolbar button tags
  • Restored correct toolbar hierarchy inside .header-right

Result

  • Eliminates duplicate DOM IDs
  • Fixes invalid nested button structure
  • Improves debugger toggle event binding reliability
  • Prevents inconsistent browser DOM auto-repair behavior
  • Improves toolbar accessibility and maintainability

Verification

Verified that:

document.querySelectorAll('#btn-debugger-toggle').length

now returns:

1

Issue

Closes #73

@siddu-k

siddu-k commented May 27, 2026

Copy link
Copy Markdown
Owner

@akash3911 has conflicts that must be resolved

@akash3911

Copy link
Copy Markdown
Contributor Author

@siddu-k once check

@siddu-k siddu-k merged commit e21fd41 into siddu-k:main May 28, 2026
1 of 2 checks passed
@akash3911 akash3911 deleted the fix/duplicate-debugger-toggle branch May 28, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Malformed Header Markup Duplicates Debugger Toggle Button

2 participants