Skip to content

fix: remove content-ui HTML bar injected on every page#33

Merged
BitHighlander merged 2 commits intodevelopfrom
hotfix/remove-injected-bar
Apr 5, 2026
Merged

fix: remove content-ui HTML bar injected on every page#33
BitHighlander merged 2 commits intodevelopfrom
hotfix/remove-injected-bar

Conversation

@BitHighlander
Copy link
Copy Markdown
Collaborator

Summary

  • Removed the content-ui script that injected an empty blue placeholder bar (bg-blue-100 div) into every web page via shadow DOM
  • Gutted content-ui/App.tsx to return null and removed all DOM injection from index.tsx
  • Removed content-ui/index.iife.js and content.css entries from manifest content_scripts

Test plan

  • Load extension, visit any website — verify no blue bar or extra DOM elements injected
  • Verify dApp injection (window.ethereum, Solana wallet standard) still works via content script
  • Check extension popup/side-panel still functions normally

🤖 Generated with Claude Code

BitHighlander and others added 2 commits April 3, 2026 15:41
The content-ui script was injecting an empty blue div (bg-blue-100) into
every web page via shadow DOM. Removed the DOM injection, gutted the App
component to return null, and removed content-ui and content.css entries
from manifest content_scripts.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The content-ui shadow DOM injection was removed, so the test
asserting its presence on example.com no longer reflects reality.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@BitHighlander BitHighlander merged commit a8ad890 into develop Apr 5, 2026
3 of 4 checks passed
BitHighlander added a commit that referenced this pull request Apr 5, 2026
Ships 5 merged bug-fix PRs: vault messaging (#34), content-ui bar
removal (#33), balance forceRefresh dedup (#36), 8 EVM handler fixes
(#35), 7 Solana handler fixes (#37).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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