Starting Touchscreen banner still exists when do nothing after launch touch screen till time out or exit calibration by ESC
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Moblin UI Framework |
Confirmed
|
Medium
|
Bob Spencer |
Bug Description
Description:
-------
Starting Touchscreen banner still exists when do nothing after launch touch screen till time out or exit calibration by ESC. Found the issue in moblin-applet 0.52, and player 0.1.0.8 in QA 20080407 RC1 image .
Reproduced steps:
-------
1. Launch touch screen from Preferences category.
2. After it prompt up calibration window, do nothing and wait for timeout.
3. Launch touch screen from Preferences category.
4. After it prompt up calibration window, press key "ESC" to exit calibration.
Current results:
-------
In step2&4, calibration window exit, but it always shows "Starting Touchscreen..." banner. It confused user that the system is launching touch screen.
Expected results:
-------
In step2&4, calibration window should exit and not show "Starting Touchscreen..." banner.
Root Cause analysis:
-------
It seems it doesn't deal with timeout correctly. Should remove "Starting Touchscreen..." banner when timeout or exit unexpectedly.
Changed in moblin-applets: | |
assignee: | nobody → todd-e-brandt |
importance: | Undecided → Medium |
ok, the problem is this, the touchscreen calibration code we inherited and that has never had troubles on samsung only looks at these five options in the xorg.conf file: MinX, MaxX, MinY, MaxY, and Rotate. The config on the crown beach apparantly adds in two rare, but valid options: SwapX and SwapY.
Let me illustrate, this is the functional config that Yang sent me (inside the touchscreen device section):
Option "MinX" "28"
Option "MinY" "46"
Option "MaxX" "991"
Option "MaxY" "954"
Option "SwapX" "1"
Option "SwapY" "1"
What this means is that the MinX, MaxX, MinY, and MaxY values look orientated typically, but the SwapX and SwapY options mean that both X and Y are actually inverted. Thus, the following should be an identical config (subtracting the two swap options):
Option "MinX" "991"
Option "MinY" "954"
Option "MaxX" "28"
Option "MaxY" "46"
Since the current touchscreen calibrator doesn't even look at the swap options, it puts the data in backwards. This is why it produces a good config when the buttons are pressed in reverse order.
Great catch Yang! I'll add a fix for 0.53 this evening.