Skip to content

[GTK] Revert unification of blank Image initialization #1891 #1893

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 12, 2025

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

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