[kwin] [Bug 466504] External touchscreen monitor maps to wrong monitor (in wayland and X11)

2023-02-27 Thread Kelvie Wong
https://bugs.kde.org/show_bug.cgi?id=466504

Kelvie Wong  changed:

   What|Removed |Added

 Resolution|--- |NOT A BUG
 Status|ASSIGNED|RESOLVED

--- Comment #4 from Kelvie Wong  ---
As mentioned in the PR comments, apparently there was a toucscreen KCM module
that allows this assignment, I just didn't think to look.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 466504] External touchscreen monitor maps to wrong monitor (in wayland and X11)

2023-02-26 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=466504

Bug Janitor Service  changed:

   What|Removed |Added

 Status|REPORTED|ASSIGNED
 Ever confirmed|0   |1

--- Comment #3 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/kwin/-/merge_requests/3705

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 466504] External touchscreen monitor maps to wrong monitor (in wayland and X11)

2023-02-26 Thread Kelvie Wong
https://bugs.kde.org/show_bug.cgi?id=466504

--- Comment #2 from Kelvie Wong  ---
Ultimately, I think we need a way for a user to choose and save the input ->
output mapping (probably a job for kscreen?) but in the meantime, perhaps we
can change the check here
(https://invent.kde.org/plasma/kwin/-/blob/6745cb87ecb57669389df3925f8e1660b8a637ad/src/backends/libinput/connection.cpp#L612-617)
:

auto testScreenMatches = [device](const Output *output) {
const auto  = device->size();
const auto  = output->physicalSize();
return std::round(size.width()) == std::round(screenSize.width())
&& std::round(size.height()) ==
std::round(screenSize.height());
};

To allow a 5% (or some other value) margin of error in the height and width?

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 466504] External touchscreen monitor maps to wrong monitor (in wayland and X11)

2023-02-26 Thread Kelvie Wong
https://bugs.kde.org/show_bug.cgi?id=466504

Kelvie Wong  changed:

   What|Removed |Added

Product|KScreen |kwin
   Assignee|kscreen-bugs-n...@kde.org   |kwin-bugs-n...@kde.org
  Component|common  |libinput

--- Comment #1 from Kelvie Wong  ---
Just took a look, and it seems like the code that assigns this is in kwin:
kwin/src/backens/libinput/connection.cpp, Connection::applyScreenToDevice

The touchscreen geometry doesn't seem to match the libinput device geometry for
this touchscreen.

For the record, libinput list-devices has the touchscreen at 309x174mm, and
xrandr has the display listed at 310x180mm (measured it and it's definitely
closer to 309x174)

-- 
You are receiving this mail because:
You are watching all bug changes.