Skip to content
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

incorrect position mapping with multiple touch screens #126

Closed
Arbel-arad opened this issue May 23, 2024 · 13 comments · Fixed by #128 or #138
Closed

incorrect position mapping with multiple touch screens #126

Arbel-arad opened this issue May 23, 2024 · 13 comments · Fixed by #128 or #138

Comments

@Arbel-arad
Copy link

when grabbing a window (with a 3-finger longpress) if the window is on a secondary display it will get teleported to the main display at the same x/y location.
i can still move it around in the mapped area and it follows the finger positions from screen 2 to screen 1.

grabbing windows on the primary screen works as intended.

@horriblename
Copy link
Owner

horriblename commented May 23, 2024

which commit? of hyprland and grass

@Arbel-arad
Copy link
Author

hyprland 0.40.0-1
hyprgrass latest version from hyprpm

this issue has been going on since i first installed hyprgrass months ago but i assumed it was caused by an unrelated hyprland bug that was fixed a few days ago.

@horriblename
Copy link
Owner

I seem to have goofed up monitor detection, send me monitor + touch screen config

@Arbel-arad
Copy link
Author

Arbel-arad commented May 23, 2024

this is my hyprland monitor config

#-- Output -----------------------------------------------------
monitor = eDP-1,preferred,0x0,1.333333,bitdepth,10
monitor = DP-3,preferred,0x1622,1.333333,bitdepth,10
monitor = ,preferred,auto,1
#-- monitor=,preferred,auto,1,mirror,DP-1

and the input settings

#-- Input: Keyboard, Mouse, Touchpad, TouchScreen, Tablet-------
input {
	numlock_by_default = true
	repeat_rate = 25
	repeat_delay = 600
	sensitivity = 0.8
	accel_profile = adaptive
	force_no_accel = false
	left_handed = false
	scroll_method = 2fg
	scroll_button = 0
	natural_scroll = false
	follow_mouse = 1
	mouse_refocus = true
	float_switch_override_focus = 0
	touchpad {
		disable_while_typing = false
		natural_scroll = true
		scroll_factor = 1.0
		clickfinger_behavior = false
		tap-to-click = true
		drag_lock = true
		tap-and-drag = true
	}
}

device {
	name = elan9008:00-04f3:2c56
	transform = 0
	output = eDP-1
}
device {
	name = elan9009:00-04f3:2c23
	transform = 0
	output = DP-3
}

device {
	name = elan9008:00-04f3:2c56-stylus
	transform = 0
	output = eDP-1
}
device {
	name = elan9009:00-04f3:2c23-stylus
	transform = 0
	output = DP-3
}

input {
    kb_layout = us,il
    kb_options = grp:alt_shift_toggle
}

@horriblename
Copy link
Owner

you have two touchscreen devices?

@Arbel-arad
Copy link
Author

yes, i have a zenbook pro duo with two touch + pen inputs.

@horriblename
Copy link
Owner

I'm surprised it works at all, is anything else broken? aside from workspace swipes. Do edge swipes work?

@Arbel-arad
Copy link
Author

Arbel-arad commented May 24, 2024

workspace swipes only work on screen 1 (no response on screen 2)
edge swipes act the same as workspace swipes
tap actions work as expected
window dragging teleports the window to screen1 (as stated above)

looks like anything that would use position information is broken but taps that act on a window are fine

@horriblename
Copy link
Owner

feel free to reopen if not fixed

@ForgotMyPasswd
Copy link

This doesn't work for me. While I don't have a device with two touchscreens, I use regularly two displays often which requires me to offset my touchscreen display if I want to place the other display above my touchscreen's display. The plugin seems to still be treating the display as if it has an offset of 0x0 even when it doesn't, with the only fix being to enter and exit sleep.

@horriblename horriblename reopened this May 29, 2024
@Arbel-arad
Copy link
Author

i tested it, looks like nothing changed.

@horriblename
Copy link
Owner

let me know if it works

I wanna make a release but I wanna first make sure this bug dies here

@Arbel-arad
Copy link
Author

i updated everything, looks like it's all working now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants