Skip to content

Fix lowlight style for dark themes - #7

Merged
san-tekart merged 1 commit into
opellen:mainfrom
PostPollux:fix-lowlight-style-for-dark-themes
Aug 11, 2026
Merged

san-tekart merged 1 commit into
opellen:mainfrom
PostPollux:fix-lowlight-style-for-dark-themes

Conversation

@PostPollux

@PostPollux PostPollux commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor

The lowlight style did not work well with dark themes. Furthermore it had a bug with grey colors that lead to a red underline.

Fixes I made:

  1. Fixed the red underline for grey colors. Now it is another shade of grey like expected.
  2. The underline color of the lowlight style now isn't anymore affected by the "auto improve readability" setting, only the background
  3. On dark themes the underline now gets actually brighter, while on light themes the behaviour is just like before, that the underline is darker.

Actually nothing changed for light themes (except the red underline fix for grey colors). But I think it looks much better now on dark themes. See the two videos for comparison:

Before:
https://github.com/user-attachments/assets/d7ddd9da-eb13-4fee-9bf9-957af193a5aa

After:
https://github.com/user-attachments/assets/0512bae3-8222-4db2-9afd-3b4f51cbd1f2

@san-tekart

Copy link
Copy Markdown
Member

Verified: typecheck clean, 157/157 tests pass. The achromatic-grey fix and the dark-theme brightening both look right, and decoupling the Lowlight underline from readability compensation is a good call — the underline needs contrast against the background, not against the theme. One tiny note: the doc comment on themedUnderline says +30pp for dark themes but the implementation (and tests) use +15pp — I'll align the comment during backport, no action needed. Squash-merging with subject fix(lowlight): theme-aware underline + achromatic grey fix (#7).

@san-tekart
san-tekart merged commit 5f1658e into opellen:main Aug 11, 2026
1 check passed
san-tekart added a commit that referenced this pull request Aug 11, 2026
Payload for the 0.1.10 marketplace release:
- manifest.json, package.json: 0.1.9 -> 0.1.10
- versions.json: add "0.1.10": "1.5.0"
- main.js: freshly rebuilt at 0.1.10

Code changes:
- fix(style): apply style body-class to all note-hosting windows instead
  of activeDocument (#8) — Obsidian 1.13 made Settings a popout window,
  so the class landed on the Settings window while notes kept the stale
  style until restart. Adds window-open re-apply for new popouts +
  4 regression tests (161/161 total).
- (already on main) fix(lowlight): theme-aware underline + achromatic
  grey fix (#7)
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