Bug 734 - Screen rotation broken under Linux
Summary: Screen rotation broken under Linux
Status: RESOLVED FIXED
Alias: None
Product: Newt
Classification: JogAmp
Component: core (show other bugs)
Version: 1
Hardware: All linux
: --- normal
Assignee: Sven Gothel
URL:
Depends on:
Blocks:
 
Reported: 2013-05-12 23:14 CEST by Julien Gouesse
Modified: 2013-07-02 17:29 CEST (History)
1 user (show)

See Also:
Type: DEFECT
SCM Refs:
3a32a59045ced338b2c5098cfb1102f847420efc
Workaround: ---

Attachments
Screen rotation buggy under Linux (37.15 KB, image/png)
2013-05-12 23:14 CEST, Julien Gouesse 		Details
Expected behaviour under Windows (58.86 KB, image/png)
2013-05-12 23:31 CEST, Julien Gouesse 		Details
unit test (8.60 KB, text/x-java)
2013-05-14 00:59 CEST, Julien Gouesse 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Gouesse 2013-05-12 23:14:51 CEST 
Created attachment 462 [details]
Screen rotation buggy under Linux

Since the latest major changes in the API (the addition of MonitorMode and the renaming of ScreenModeUtil -> MonitorModeUtil), the screen rotation is partially broken under Linux whereas it still works under Windows. I use an available MonitorMode instance whose rotation is set to 90 degrees as you can see in the screen captures. As you can see, the mouse pointer is correctly rotated but not the rest of the GUI.
Comment 1 Julien Gouesse 2013-05-12 23:31:35 CEST 
Created attachment 463 [details]
Expected behaviour under Windows
Comment 2 Julien Gouesse 2013-05-14 00:59:47 CEST 
Created attachment 464 [details]
unit test
Comment 3 Julien Gouesse 2013-05-14 01:00:39 CEST 
I've just added a unit test. The bug is reproducible only when the highest available resolution is used and in fullscreen mode.
Comment 4 Sven Gothel 2013-07-02 16:39:27 CEST 
Could not confirm (see commit 3a32a59045ced338b2c5098cfb1102f847420efc),
however .. maybe previous fixes solved this problem.

Valid bug - marked resolved, since investigation of this triggered review of 
fullscreen and mode change behavior, i.e. see Bug 770 and Bug 771.
Comment 5 Julien Gouesse 2013-07-02 17:29:45 CEST 
(In reply to comment #4)
> Could not confirm (see commit 3a32a59045ced338b2c5098cfb1102f847420efc),
> however .. maybe previous fixes solved this problem.
> 
> Valid bug - marked resolved, since investigation of this triggered review of 
> fullscreen and mode change behavior, i.e. see Bug 770 and Bug 771.

As soon as the unit test passes, it is ok for me.