Skip to content

Conversation

@rinormaloku
Copy link
Contributor

@rinormaloku rinormaloku commented Jan 21, 2026

This plugin enables users to set the threshold when something is registered as a click vs a drag.

@gonX
Copy link
Member

gonX commented Jan 21, 2026

Download and repo link both returns 404 at the moment. Make sure the repository is set to public.

@rinormaloku
Copy link
Contributor Author

Rookie mistake, now it is public

Copy link
Member

@gonX gonX left a comment

Choose a reason for hiding this comment

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

Code in repo looks fine

@gonX gonX requested a review from Kuuuube January 21, 2026 20:10
@Kuuuube Kuuuube added the addition New repository entry label Jan 21, 2026
@Kuuuube
Copy link
Member

Kuuuube commented Jan 21, 2026

I'm fine with the plugin but I think you should change the description. I like your description on the repo a lot more.

Prevents accidental cursor movement during pen clicks by defining a movement threshold that must be exceeded before cursor movement is registered as a drag

The description you've put here could be pretty confusing Adjusts the drag threshold for tablet input.

Could make it a bit shorter if you want but an actual description of the plugin function is better because users may not know what a "drag threshold" means.

@rinormaloku
Copy link
Contributor Author

Done!

btw I removed the smooth option, as it would work only if someone uses the wacom tablet in relative mode, which nobody does, so sort of confusing and useless.

@Kuuuube Kuuuube merged commit b7355ed into OpenTabletDriver:master Jan 22, 2026
1 check passed
@Nyunyun
Copy link

Nyunyun commented Jan 23, 2026

I'm just commenting here that it seems that the plugin gets installed incorrectly.

$ tree ~/.config/OpenTabletDriver/Plugins/Drag\ Threshold/
/home/nyunyun/.config/OpenTabletDriver/Plugins/Drag Threshold/
├── DragThreshold
│   ├── DragThreshold.deps.json
│   └── DragThreshold.dll
└── metadata.json

The .json and .dll should be in the root directory of Drag Treshold.

@Kuuuube
Copy link
Member

Kuuuube commented Jan 23, 2026

Will yank from the repo if this is not fixed reasonably soon. Please make another PR updating to the corrected release assets @rinormaloku.

For future reference, you can test if your plugin installs correctly from plugin manager after pushing changes to your fork of the plugin repo by going to File > Use alternate source, entering your details (in this case you would only need to set the repo owner to yourself), and installing the plugin from the list. You can also verify that your zip file works simply by loading in the zip through File > Install plugin... or drag and drop.

@rinormaloku
Copy link
Contributor Author

I'll take a look right away

@rinormaloku
Copy link
Contributor Author

@Kuuuube here the fix: #144

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

addition New repository entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants