Skip to content

feat: propagate host UI theme into OnlyOffice loader#16

Merged
cs50victor merged 3 commits intomainfrom
fix/issue-585-office-theme-persistence
Mar 30, 2026
Merged

feat: propagate host UI theme into OnlyOffice loader#16
cs50victor merged 3 commits intomainfrom
fix/issue-585-office-theme-persistence

Conversation

@interpreterwork
Copy link
Copy Markdown
Collaborator

@interpreterwork interpreterwork commented Mar 21, 2026

Summary

  • accept theme on /open and forward it to the offline loader URL
  • map light|dark|system to OnlyOffice UI themes in offline-loader-proper.html
  • initialize desktop stub theme state from the query theme instead of hardcoded light

Verification

  • cd oo-editors && bun run test:unit
  • Manual route check:
    • started oo-editors server with FONT_DATA_DIR=assets/onlyoffice-fontdata
    • requested /open?filepath=/tmp/demo.docx&theme=dark&lang=en
    • verified redirect Location includes theme=dark

@cs50victor cs50victor changed the title Propagate host UI theme into OnlyOffice loader feat: propagate host UI theme into OnlyOffice loader Mar 30, 2026
@cs50victor cs50victor merged commit 2ebe53f into main Mar 30, 2026
4 checks passed
@cs50victor cs50victor deleted the fix/issue-585-office-theme-persistence branch March 30, 2026 23:31
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.

2 participants