Skip to content

fix(wikipedia): recolor {{text diff}} template - #1953

Open
salarua wants to merge 2 commits into
catppuccin:mainfrom
salarua:wikipedia-fix-text-diff
Open

fix(wikipedia): recolor {{text diff}} template#1953
salarua wants to merge 2 commits into
catppuccin:mainfrom
salarua:wikipedia-fix-text-diff

Conversation

@salarua

@salarua salarua commented Oct 7, 2025

Copy link
Copy Markdown
  • Change --background-color-content-removed and --background-color-content-added vars to use Catppuccin colors
  • Add rule recoloring .diffchange color to --color-inverted for visibility on colored background; uses !important because every .diffchange element is individually styled inline

馃敡 What does this fix? 馃敡

Closes #1951

馃棐 Checklist 馃棐

* Change --background-color-content-removed and
--background-color-content-added vars to use Catppuccin colors
* Add rule recoloring .diffchange color to --color-inverted for visibility on
colored background; uses !important because every .diffchange element is
individually styled inline

Fixes: catppuccin#1951
@salarua
salarua requested a review from uncenter as a code owner October 7, 2025 22:13
@github-actions github-actions Bot added the wikipedia Wikipedia label Oct 7, 2025
@salarua

salarua commented Oct 7, 2025

Copy link
Copy Markdown
Author

Preview images

Light theme

Unstyled
Screenshot_20251007_152405

v2025.06.04
Screenshot_20251007_152654

v2025.10.07
Screenshot_20251007_152749

Dark theme (Mocha)

Unstyled
Screenshot_20251007_152634

v2025.06.04
Screenshot_20251007_152721

v2025.10.07
Screenshot_20251007_152800

Comment thread styles/wikipedia/catppuccin.user.less Outdated
@uncenter

uncenter commented Oct 8, 2025

Copy link
Copy Markdown
Member

Blocked by #1867 (and #1866).

@uncenter uncenter added the 2.status: pending Pending until internal discussion, planned updates, etc. label Oct 8, 2025
@uncenter uncenter removed the 2.status: pending Pending until internal discussion, planned updates, etc. label Dec 29, 2025
@uncenter

Copy link
Copy Markdown
Member

My apologies for the delay. I think we can get this in without waiting for those blocking changes.

Comment on lines +170 to +171
--background-color-content-added: @blue;
--background-color-content-removed: @yellow;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Instead of adding the override for the text color below, we can fade the diff background color to 10-20% as per the style guide (https://github.com/catppuccin/catppuccin/blob/main/docs/style-guide.md#code-editors) and then keep the regular foreground color.

@WalkQuackBack WalkQuackBack added 0.kind: bug Bug (i.e. unthemed components) 2.status: changes-requested Changes requested by reviewers must be attended to by author before further review labels Jan 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.kind: bug Bug (i.e. unthemed components) 2.status: changes-requested Changes requested by reviewers must be attended to by author before further review wikipedia Wikipedia

Projects

None yet

Development

Successfully merging this pull request may close these issues.

{{text diff}} template is unreadable

3 participants