SpreadSheet Editor Dark Mode (dark document) 🌘 — companion to sdkjs PR - #187
Open
j-base64 wants to merge 11 commits into
Open
SpreadSheet Editor Dark Mode (dark document) 🌘 — companion to sdkjs PR#187j-base64 wants to merge 11 commits into
j-base64 wants to merge 11 commits into
Conversation
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
…sdkjs commit too Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
…ark-mode correction) Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Clean up dark-mode toggle debounce comment in ViewTab.js Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
…theme switches Signed-off-by: j-base64 <jcentenero@arsys.es> Assisted-by: ClaudeCode:claude-sonnet-5
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Origin
Issue #117 pinpoints the lack of dark mode in the spreadsheet editor, unlike the document editor, which already has one.
Originally noticed on web-apps, the fix requires changes to both
sdkjsandweb-apps, plus optionallyDocumentServerfor e2e smoke tests.Approach
This PR brings dark mode (dark document feature) to the spreadsheet editor with a simple starting approach, rather than full negative-inversion or full token-based re-theming:
SSE-DarkMode_ScreenCapture_300729.mp4
What this dark mode DOES ✅
Design notes 🌈
What this dark mode DOESN'T DO ❌
Known tradeoffs
¹ Automatic color = different from explicit color (those explicitly set by the user or a table style template)
Basic manual testing
Viewtab > Make sure the interface theme is Dark.Dark Documentbutton to see differences.Automated testing
DocumentServersee below, 13 smoke tests covering cell fill/border/text correction, merged cells, page-break-preview, gradient/pattern fill in the cell editor, print-preview, column-resize guide color, search-highlight contrast, mid-edit-toggle behavior, interface-theme interaction, and toolbar syncCode
This PR spans BOTH
sdkjs,web-apps, and optionallyDocumentServer:sdkjsPR: SpreadSheet Editor Dark Mode (dark document) 🌘 — companion to web-apps PR sdkjs#70.DocumentServercompanion PR for e2e/smoketests: SpreadSheet Editor Dark Mode (dark document) 🌘 — e2e smoke tests DocumentServer#310.AI assistance
claude-sonnet-5)