Skip to content

Conversation

@HeikoKlare
Copy link
Contributor

A previous change unified the Image surface initialization for blank Image instances just based on height and width. Because of the change, screen capturing using a GC on the display does only work properly when executed the first time in an application lifecycle and in specific situation, the initial properties of a GC changed (such as the text anti-aliasing behavior). Due to a lack of a better solution for now, this change reverts the unification of the Image surface initialization to the previous state.

Fixes #1891

This partially reverts commit 5c2611d.

Supercedes and thus closes #1835
Supercedes and thus closes #1839

@github-actions
Copy link
Contributor

github-actions bot commented Mar 10, 2025

Test Results

   510 files  + 1     510 suites  +1   8m 59s ⏱️ -42s
 4 345 tests +37   4 331 ✅ +35   14 💤 +3  0 ❌  - 1 
16 613 runs  +37  16 502 ✅ +35  111 💤 +3  0 ❌  - 1 

Results for commit e50db6b. ± Comparison against base commit 487d1cb.

♻️ This comment has been updated with latest results.

@HeikoKlare HeikoKlare marked this pull request as ready for review March 11, 2025 11:30
…rm#1891

A previous change unified the Image surface initialization for blank
Image instances just based on height and width. Because of the change,
screen capturing using a GC on the display does only work properly when
executed the first time in an application lifecycle and in specific
situation, the initial properties of a GC changed (such as the text
anti-aliasing behavior). Due to a lack of a better solution for now,
this change reverts the unification of the Image surface initialization
to the previous state.

Fixes eclipse-platform#1891

This partially reverts commit 5c2611d.
@HeikoKlare HeikoKlare merged commit 7f381e8 into eclipse-platform:master Mar 12, 2025
14 checks passed
@HeikoKlare HeikoKlare deleted the issue-1891 branch March 12, 2025 10:23
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.

SWT Linux Gtk - Capturing screenshots broken in 4.35

1 participant