Full touch api integration, rip drag events
Simplify conditional statement
Full touch api integration, rip drag events
Simplify conditional statement
No Linters Available |
No Unit Test Coverage |
Hi,
I have a small request before i review and test it.
The code has some commented code in it, a redundant QTimer include (in the header where it also forward declares it) and a commented todo. I'm guessing you still need to do some cleanup?
Also, the summary is a bit misleading. You say "rip drag events" which lead me to think that drag/drop is completely removed. That doesn't seem to be the case when looking at the code as you haven't deleted those event handlers (thankfully, drag/drop needs to stay obviously).
Lastly, how do i test touch events on a non-touch monitor? Is there some kind of touch emulation for the desktop?
And @markg I found this tool that lets you simulate a touchscreen: https://github.com/vi/virtual_touchscreen
Hmm, that might. That's darn tricky to spot if you're not looking for it ;)
And thank you for the virtual touchscreen link, will have a look at that.