Bug 515745 - Negative screen coordinates lead to failed tests (Version: 8.4.1.123)
Summary: Negative screen coordinates lead to failed tests (Version: 8.4.1.123)
Status: CLOSED FIXED
Alias: None
Product: Jubula (Archived)
Classification: Technology
Component: RC (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 10
: P3 blocker (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact: Elena Pister CLA
URL:
Whiteboard: stalebug
Keywords: helpwanted
Depends on: 367156
Blocks:
  Show dependency tree
 
Reported: 2017-04-25 05:19 EDT by Marcel Schramm CLA
Modified: 2021-06-29 06:03 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel Schramm CLA 2017-04-25 05:19:24 EDT
The problem:

When executing tests, while having the AUT on the right screen (Screen 1), everything goes fine, but as soon as the aut is on my left screen (Screen 2) the text execution fails due to negative coordinates. That doesn't make any sense, because negative coordinates don't imply that they are aren't correct.
When having the right screen as screen 1 and the left screen as screen 2, the x coordiante 0:0 is at the top left corner of screen 1, as shown in the example below:

-1280         0       1279
    -------------------
    |        |        |
    |   2    |   1    |
    |        |        |
    -------------------

The same problem might occur with the y-coordinates, depending on the screen setup.
Comment 1 Marcel Schramm CLA 2017-04-25 05:38:35 EDT
I know, that this has been asked of a while ago, but still, i'd like to point this out again, maybe you have changed your mind.
Comment 2 Marvin Mueller CLA 2017-06-26 08:03:49 EDT
Thanks for the Information, since multi monitor setups are getting more and more frequent, I agree that this feature indeed might be useful. But at this moment we have not enough resources to test this and find all places where checks like this(mentioned in 367156) are implemented.

Maybe we could remove easily some of the checks and look what happens.

If someone wants to make a contribution we will happy to review it.
Comment 3 Miklos Hartmann CLA 2017-07-12 16:47:51 EDT
I did a quick check of the code, and I think there is a single place where we check against negative coordinates, and funnily, this only in RobotSwtImpl. I think other places in the code are fairly neutral with respect to negaive coordinates, so it might worth simply removing this artificial check. Or before that it may worth testing what happens in a Swing / JavaFX AUT if we use multiple screens. Unfortunately I only have a single monitor, so I cannot myself test this...
Comment 4 Miklos Hartmann CLA 2017-07-17 05:37:16 EDT
Contribution: https://git.eclipse.org/r/#/c/101324/

This contribution removes the extra check from the SWT / RCP tests. In theory this should allow to use multiple screens to the same extent as Swing / JavaFX allows.

I'm not setting the ticket Resolved yet, because I'm not able to properly test the above assumption.
Comment 6 Eclipse Genie CLA 2019-07-08 03:23:25 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 7 Eclipse Genie CLA 2021-06-28 10:14:53 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 8 Marvin Mueller CLA 2021-06-29 06:03:42 EDT
We have removed most of the checks with the Screen. If there ist still such errors please reopen the bug