Skip to content

[GTK4] Set source color when doing difference on pixels in Caret#2813

Closed
jonahgraham wants to merge 1 commit into
eclipse-platform:masterfrom
jonahgraham:set-caret-source-color
Closed

[GTK4] Set source color when doing difference on pixels in Caret#2813
jonahgraham wants to merge 1 commit into
eclipse-platform:masterfrom
jonahgraham:set-caret-source-color

Conversation

@jonahgraham
Copy link
Copy Markdown
Contributor

With no source color explicitly set here, the difference is applied using whatever happens to have been the last set source color, or 0,0,0 if none have been set.

See #2812 (comment) for a longer explanation.

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

  118 files  ±0    118 suites  ±0   16m 15s ⏱️ ±0s
4 653 tests ±0  4 636 ✅ ±0  17 💤 ±0  0 ❌ ±0 
  338 runs  ±0    334 ✅ ±0   4 💤 ±0  0 ❌ ±0 

Results for commit 4b84b72. ± Comparison against base commit a0d6ffc.

@akurtakov
Copy link
Copy Markdown
Member

Is this one still needed?

With no source color explicitly set here, the difference is applied
using whatever happens to have been the last set source color, or 0,0,0
if none have been set.

See eclipse-platform#2812 (comment)
for a longer explanation.
@akurtakov akurtakov force-pushed the set-caret-source-color branch from 4b84b72 to 707e52b Compare June 1, 2026 19:14
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

Test Results (linux)

   94 files  +   31     94 suites  +31   14m 57s ⏱️ + 6m 35s
4 603 tests ±    0  4 379 ✅ ±    0  224 💤 ± 0  0 ❌ ±0 
3 401 runs  +1 125  3 326 ✅ +1 084   75 💤 +41  0 ❌ ±0 

Results for commit 707e52b. ± Comparison against base commit 9ab3898.

@akurtakov
Copy link
Copy Markdown
Member

akurtakov commented Jun 2, 2026

I believe #3356 is a more complete fix than this PR.

@jonahgraham
Copy link
Copy Markdown
Contributor Author

Using CAIRO_OPERATOR_OVER as done in #3356 makes sense, so closing this in favour of #3356.

@jonahgraham jonahgraham closed this Jun 2, 2026
@jonahgraham jonahgraham deleted the set-caret-source-color branch June 2, 2026 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants