Skip to content

Disable printscreen key #1

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.

Sign up for GitHub

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

Closed
wants to merge 3 commits into from
Closed

Disable printscreen key #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
12ac449
[GTK4] Do not mix gtk 3 and 4 focus event handlers
akurtakov Oct 2, 2025
72ae23c
Screenshot
HeikoKlare Oct 2, 2025
ab64626
Turn off prtscr test
jonahgraham Oct 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .github/workflows/maven.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,3 +99,9 @@ jobs:
path: |
${{ github.workspace }}/**/target/surefire-reports/*.xml
${{ github.workspace }}/**/hs_err_pid*.log
- name: Upload screenshots
if: always()
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
with:
name: screenshots
path: tests/org.eclipse.swt.tests.win32/screenshots
4 changes: 2 additions & 2 deletions binaries/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,13 @@
</properties>

<modules>
<module>org.eclipse.swt.cocoa.macosx.x86_64</module>
<!--<module>org.eclipse.swt.cocoa.macosx.x86_64</module>
<module>org.eclipse.swt.cocoa.macosx.aarch64</module>
<module>org.eclipse.swt.gtk.linux.aarch64</module>
<module>org.eclipse.swt.gtk.linux.ppc64le</module>
<module>org.eclipse.swt.gtk.linux.riscv64</module>
<module>org.eclipse.swt.gtk.linux.x86_64</module>
<module>org.eclipse.swt.win32.win32.aarch64</module>
<module>org.eclipse.swt.win32.win32.aarch64</module>-->
<module>org.eclipse.swt.win32.win32.x86_64</module>
</modules>

Expand Down
17 changes: 9 additions & 8 deletions bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Canvas.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,14 +260,15 @@ long gtk_focus_out_event (long widget, long event) {
}

@Override
void gtk4_focus_window_event(long handle, long event) {
if(event == SWT.FocusIn) {
gtk_focus_in_event (handle, event);
if (caret != null) caret.setFocus ();
} else {
gtk_focus_out_event(handle, event);
if (caret != null) caret.killFocus();
}
void gtk4_focus_enter_event(long handle, long event) {
super.gtk4_focus_enter_event (handle, event);
if (caret != null) caret.setFocus ();
}

@Override
void gtk4_focus_leave_event(long handle, long event) {
super.gtk4_focus_leave_event (handle, event);
if (caret != null) caret.setFocus ();
}

@Override
Expand Down
12 changes: 6 additions & 6 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,15 +183,15 @@


<modules>
<module>bundles</module>
<module>binaries</module>
<module>bundles/org.eclipse.swt</module>
<module>binaries/org.eclipse.swt.win32.win32.x86_64</module>
<module>local-build/org.eclipse.swt.fragments.localbuild</module>
<module>examples/org.eclipse.swt.examples</module>
<!--<module>examples/org.eclipse.swt.examples</module>
<module>examples/org.eclipse.swt.examples.browser.demos</module>
<module>examples/org.eclipse.swt.examples.launcher</module>
<module>examples/org.eclipse.swt.examples.ole.win32</module>
<module>examples/org.eclipse.swt.examples.views</module>
<module>tests/org.eclipse.swt.tests</module>
<module>features/org.eclipse.swt.tools.feature</module>
<module>examples/org.eclipse.swt.examples.views</module>-->
<module>tests/org.eclipse.swt.tests.win32</module>
<!--<module>features/org.eclipse.swt.tools.feature</module>-->
</modules>
</project>
21 changes: 20 additions & 1 deletion ...g.eclipse.swt.tests.win32/JUnit Tests/org/eclipse/swt/tests/win32/KeyboardLayoutTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,16 @@

import static org.junit.jupiter.api.Assertions.fail;

import java.io.File;
import java.io.IOException;
import java.nio.file.Files;
import java.util.ArrayList;

import org.eclipse.swt.SWT;
import org.eclipse.swt.graphics.GC;
import org.eclipse.swt.graphics.Image;
import org.eclipse.swt.graphics.ImageData;
import org.eclipse.swt.graphics.ImageLoader;
import org.eclipse.swt.internal.win32.INPUT;
import org.eclipse.swt.internal.win32.KEYBDINPUT;
import org.eclipse.swt.internal.win32.OS;
Expand Down Expand Up @@ -221,9 +228,21 @@ public boolean isExtended() {
};

@BeforeEach
public void setUp(TestInfo testInfo) {
public void setUp(TestInfo testInfo) throws IOException {
this.testName = testInfo.getDisplayName();
display = new Display();
GC gc = new GC(display);
Image image = new Image(display, display.getBounds().width, display.getBounds().height);
gc.copyArea(image, 0, 0);
ImageLoader loader = new ImageLoader();
loader.data = new ImageData[] {image.getImageData() };
File folder = new File("screenshots");
if (!folder.exists()) {
Files.createDirectory(folder.toPath());
}
String filePath = folder.getAbsolutePath();
loader.save(filePath + "/" + testName + ".png", SWT.IMAGE_PNG);

shell = new Shell();

Listener listener = event -> {
Expand Down
16 changes: 7 additions & 9 deletions ...s.win32/JUnit Tests/org/eclipse/swt/tests/win32/Test_org_eclipse_swt_events_KeyEvent.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,13 @@
import org.eclipse.swt.widgets.Event;
import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.condition.DisabledIfEnvironmentVariable;

/**
* Automated Test Suite for class org.eclipse.swt.events.KeyEvent
*
* @see org.eclipse.swt.events.KeyEvent
*/
@SuppressWarnings("restriction")
@DisabledIfEnvironmentVariable(named = "GITHUB_ACTIONS", matches = "true", disabledReason = "Windows Server 2025 incompatibility: https://github.com/eclipse-platform/eclipse.platform.swt/issues/2516")
public class Test_org_eclipse_swt_events_KeyEvent extends KeyboardLayoutTest {
// Windows layouts suitable for 'LoadKeyboardLayout()', obtained from 'GetKeyboardLayoutName()'
static char[] LAYOUT_BENGALI = "00020445\0".toCharArray();
Expand Down Expand Up @@ -236,7 +234,7 @@ public void testEnglishUs_stdKeys() {
new KeyDescription(UsScan.F8, SWT.F8, '\0', '\0' ),
new KeyDescription(UsScan.F9, SWT.F9, '\0', '\0' ),
new KeyDescription(UsScan.F10, SWT.F10, '\0', '\0' ),
new KeyDescription(UsScan.PrtScr, SWT.PRINT_SCREEN,'\0', '\0' ),
// new KeyDescription(UsScan.PrtScr, SWT.PRINT_SCREEN,'\0', '\0' ),
new KeyDescription(UsScan.Oem102, '\\', '\\', '|' ),
new KeyDescription(UsScan.F11, SWT.F11, '\0', '\0' ),
new KeyDescription(UsScan.F12, SWT.F12, '\0', '\0' ),
Expand Down Expand Up @@ -292,12 +290,12 @@ public void testEnglishUs_stdKeys() {
if (testKey.scanCode == UsScan.PrtScr) {
// For some reason, PrintScreen only produces SWT.KeyUp event.
// Maybe because Windows handles WM_KEYDOWN to copy screen to clipboard?
String testName = getKeyName(state, testKey.scanCode);
expectKeyEvents(
testName,
() -> emulateScanCode(state, testKey.scanCode),
expectKeyUp(state, expectedChar, 0, testKey.keyCode)
);
// String testName = getKeyName(state, testKey.scanCode);
// expectKeyEvents(
// testName,
// () -> emulateScanCode(state, testKey.scanCode),
// expectKeyUp(state, expectedChar, 0, testKey.keyCode)
// );
continue;
}

Expand Down
Loading