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

Added Touchpad Config options #200

Merged
merged 4 commits into from
Jun 13, 2022
Merged

Added Touchpad Config options #200

merged 4 commits into from
Jun 13, 2022

Conversation

siemato
Copy link
Contributor

@siemato siemato commented Jun 12, 2022

Added a bunch of options for clickpads (disable tap-to-click, disable middle mouse emulation and enable clickfinger behavior). #199 can be closed on merge.

Added Options to 1. toggle between clickfinger behavior and software buttons, 2. middle button emulation and 3. tap-to-click
Implemented Options to 1. toggle between clickfinger behavior and software buttons, 2. button emulation and 3. tap-to-click
Copy link
Member

@fufexan fufexan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Maybe you can get rid of the verbose == by inversing the conditions, but that's just my preference.

@vaxerski
Copy link
Member

code styling. either take what @fufexan said, or

if(somefunc() == 1)

without spacing it looks cluttered.

@siemato
Copy link
Contributor Author

siemato commented Jun 12, 2022

Fixed, please have a look and merge

@vaxerski
Copy link
Member

i will wait for freedesktop gitlab to reincarnate itself for the checks to succeed

@ThatOneCalculator
Copy link
Contributor

cc @vaxerski

@vaxerski vaxerski merged commit a4e21a2 into hyprwm:main Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants