lhumina_code/hero_aibroker
23
0
Fork 0
Code Issues 3 Pull requests Projects Releases 1 Packages Wiki Activity Actions

Phase 3: Iframe theme sync — remove hardcoded dark-mode styles #18

Closed
mik-tf wants to merge 2 commits from phase3-theme-sync into development
pull from: phase3-theme-sync
merge into: lhumina_code:development
Conversation 0 Commits 2 Files changed 6 +70 -69
mik-tf commented 2026-03-07 05:13:36 +00:00
Owner
Copy link

Closes #17

Summary

Remove all hardcoded dark-mode Bootstrap classes and hex colors from admin UI templates. Add bsMap theme receiver for hero:theme postMessage protocol.

Changes

  • 6 template files modified, 86 insertions, 70 deletions
  • All bg-dark → bg-body-secondary, text-white → text-body
  • Removed data-bs-theme="dark" (receiver JS controls it)
  • Hex colors replaced with CSS vars (var(--bs-body-bg), var(--bs-border-color))

Test plan

  • Toggle light/dark mode in Hero OS Settings
  • Verify all AI Broker tabs follow theme
  • Check chat pane, models modal, logs table
Closes #17 ## Summary Remove all hardcoded dark-mode Bootstrap classes and hex colors from admin UI templates. Add bsMap theme receiver for hero:theme postMessage protocol. ## Changes - 6 template files modified, 86 insertions, 70 deletions - All `bg-dark` → `bg-body-secondary`, `text-white` → `text-body` - Removed `data-bs-theme="dark"` (receiver JS controls it) - Hex colors replaced with CSS vars (`var(--bs-body-bg)`, `var(--bs-border-color)`) ## Test plan - [ ] Toggle light/dark mode in Hero OS Settings - [ ] Verify all AI Broker tabs follow theme - [ ] Check chat pane, models modal, logs table
Phase 3: Iframe theme sync — bsMap receiver + dark-mode class removal
All checks were successful
Build and Test / build-and-test (pull_request) Successful in 2m1s
Details
10017a5375 
Added hero:theme postMessage receiver with Bootstrap var mapping (bsMap)
to sync Hero OS theme to iframe admin UIs. Removed hardcoded dark-mode
classes (bg-dark, text-white, navbar-dark, table-dark) and replaced with
theme-aware Bootstrap 5.3 equivalents (bg-body-secondary, text-body).
Removed hardcoded data-bs-theme="dark" so receiver JS controls it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Phase 3: Add hero-bootstrap-bridge.css — single source of truth
All checks were successful
Build and Test / build-and-test (pull_request) Successful in 1m57s
Details
45408a19c6 
Replace per-template bsMap JS with shared CSS file served by hero_proxy.
The bridge CSS maps Hero OS --color-* vars to Bootstrap --bs-* vars and
includes safety-net overrides for bg-dark, navbar-dark, table-dark in
light mode. Receiver JS simplified to just set vars + data-bs-theme.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
mik-tf closed this pull request 2026-03-07 05:31:45 +00:00
All checks were successful
Build and Test / build-and-test (pull_request) Successful in 1m57s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
prio_critical

   
prio_low

   
type_bug

   
type_contact

   
type_issue

   
type_lead

   
type_question

   
type_story

   
type_task
No labels
prio_critical
prio_low
type_bug
type_contact
type_issue
type_lead
type_question
type_story
type_task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_aibroker!18
No description provided.