SpreadSheet Editor Dark Mode (dark document) π β companion to web-apps PR - #70
Open
j-base64 wants to merge 20 commits into
Open
SpreadSheet Editor Dark Mode (dark document) π β companion to web-apps PR#70j-base64 wants to merge 20 commits into
j-base64 wants to merge 20 commits into
Conversation
β¦web-apps commit too Assisted-by: ClaudeCode:claude-sonnet-5 Signed-off-by: j-base64 <jcentenero@arsys.es>
β¦mode correction) 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 Signed-off-by: j-base64 <jcentenero@arsys.es>
- disable isNearBlackOrWhite computation 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 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 Signed-off-by: j-base64 <jcentenero@arsys.es>
Assisted-by: ClaudeCode:claude-sonnet-5 Signed-off-by: j-base64 <jcentenero@arsys.es>
β¦rk mode 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 Signed-off-by: j-base64 <jcentenero@arsys.es>
β¦ computes (no behavior change) 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>
Fix dark-mode cell color bugs, theme-corruption bug, and naming cleanup - Fix page-break-preview border overlay getting dark-mode corrected twice - Fix pattern-fill contrast check treating a known cell-editor background as unknown - Fix interface theme switches silently corrupting content dark mode's cell background/grid color (GlobalSkin aliases directly into EditorSkins; the interface-skin color harvest was overwriting it in place) - Rename bIsExplicitFill/isNotCustomColor/misapplied isTextRecolorable locals to match what they actually compute (no behavior change) - Remove redundant/inaccurate comments introduced by this branch Assisted-by: ClaudeCode:claude-sonnet-5 Signed-off-by: j-base64 <jcentenero@arsys.es>
Tune isColorDark threshold and drop a same-session comment 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 Signed-off-by: j-base64 <jcentenero@arsys.es>
j-base64
marked this pull request as ready for review
August 4, 2026 09:34
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 Euro-Office/web-apps#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:web-appsPR for the UI part: SpreadSheet Editor Dark Mode (dark document) π β companion to sdkjs PRΒ web-apps#187.DocumentServercompanion PR for e2e/smoketests: SpreadSheet Editor Dark Mode (dark document) π β e2e smoke testsΒ DocumentServer#310.AI assistance
claude-sonnet-5)