-
Notifications
You must be signed in to change notification settings - Fork 119
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
#6 [win32] Keyboard shortcuts for radio buttons don't work anymore #24
Conversation
… work anymore Signed-off-by: Rolf Theunissen <rolf.theunissen@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verified on Win10.
Thanks Rolf for the fix.
@@ -425,6 +428,48 @@ public void test_setTextLjava_lang_String() { | |||
button.setText(""); | |||
} | |||
|
|||
@Test | |||
public void test_traverseCheckButton() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test fails on Linux.
https://download.eclipse.org/eclipse/downloads/drops4/I20220406-0600/testresults/html/org.eclipse.swt.tests_ep424I-unit-cen64-gtk3-java11_linux.gtk.x86_64_11.html
Test_org_eclipse_swt_widgets_Button | test_traverseCheckButton | Failure | N/A
java.lang.AssertionErrorat org.junit.Assert.fail(Assert.java:87)
at org.junit.Assert.assertTrue(Assert.java:42)
at org.junit.Assert.assertTrue(Assert.java:53)
at org.eclipse.swt.tests.junit.Test_org_eclipse_swt_widgets_Button.test_traverseCheckButton(Test_org_eclipse_swt_widgets_Button.java:448)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@niraj-modi Did you check the build results before merging?
I was not able to check them because build results were removed already, probably as result of the merge. I wonder if the test consistently fails or if it is flacky.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tried it on Linux locally, that was failing for me two times in the row.
@@ -425,6 +428,48 @@ public void test_setTextLjava_lang_String() { | |||
button.setText(""); | |||
} | |||
|
|||
@Test | |||
public void test_traverseCheckButton() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test fails on Linux.
https://download.eclipse.org/eclipse/downloads/drops4/I20220406-0600/testresults/html/org.eclipse.swt.tests_ep424I-unit-cen64-gtk3-java11_linux.gtk.x86_64_11.html
Test_org_eclipse_swt_widgets_Button | test_traverseCheckButton | Failure | N/A
java.lang.AssertionErrorat org.junit.Assert.fail(Assert.java:87)
at org.junit.Assert.assertTrue(Assert.java:42)
at org.junit.Assert.assertTrue(Assert.java:53)
at org.eclipse.swt.tests.junit.Test_org_eclipse_swt_widgets_Button.test_traverseCheckButton(Test_org_eclipse_swt_widgets_Button.java:448)
… work anymore (eclipse-platform#24) Change-Id: I09f81bc5041bb1aa61fb7b62300a772aa6853abf Signed-off-by: Rolf Theunissen <rolf.theunissen@gmail.com>
Change-Id: I09f81bc5041bb1aa61fb7b62300a772aa6853abf Signed-off-by: Rolf Theunissen <rolf.theunissen@gmail.com>
* #6 [win32] Keyboard shortcuts for radio buttons don't work anymore (#24) Change-Id: I09f81bc5041bb1aa61fb7b62300a772aa6853abf Signed-off-by: Rolf Theunissen <rolf.theunissen@gmail.com> * Bug 579626 - RTL text is not rendered correctly on TextLayout #37 Change-Id: I74f0fc34159d2ae6442b501eb2e8038c82c03ac2 Signed-off-by: Niraj Modi <niraj.modi@in.ibm.com> Co-authored-by: rolfth <rolf.theunissen@gmail.com>
Signed-off-by: Rolf Theunissen rolf.theunissen@gmail.com