Skip to content

Conversation

@JammingBen
Copy link
Contributor

And removes the light/dark theme switch within Collabora.

closes #705

@JammingBen JammingBen self-assigned this Oct 14, 2025
@JammingBen JammingBen marked this pull request as ready for review October 14, 2025 13:48
Copilot AI review requested due to automatic review settings October 14, 2025 13:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements light/dark theme support for Collabora integration by respecting the current theme setting and removing the internal theme switch within Collabora.

  • Adds theme detection and UI defaults parameter to pass theme preference to Collabora
  • Hides Collabora's internal dark theme toggle button via post message
  • Updates test suite to verify theme parameter is correctly set based on current theme

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/web-app-external/src/App.vue Implements theme detection, adds UI defaults parameter, and hides Collabora's theme toggle
packages/web-app-external/tests/unit/app.spec.ts Adds comprehensive tests for theme functionality and updates test helper function
packages/web-app-external/tests/unit/snapshots/app.spec.ts.snap Updates snapshot to reflect new hidden input field

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@AlexAndBear AlexAndBear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍🌝

@JammingBen JammingBen merged commit 366b049 into main Oct 14, 2025
29 checks passed
@JammingBen JammingBen deleted the feat/collabora-dark-theme-support branch October 14, 2025 15:20
@openclouders openclouders mentioned this pull request Oct 14, 2025
1 task
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.

Collabora: Use dark/light mode automatically based on user preference

3 participants