-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
More tap gestures #117
More tap gestures #117
Conversation
Mac & Linux are failing because GTK3. But why is Windows succeeding |
The Windows CI doesn't build WinUIBackend at the moment due to some annoying Swift on Windows issues. We can't use anything newer than Swift 5.10 cause of some weird issue where SwiftPM fails to compile the manifest due to missing CPP headers, but swift-winui requires Swift 6.0 |
Gtk3 and WinUI still only actually support .primary, but at least now they'll compile.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for these improvements! 🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops I meant to request changes instead of approve hahah, just two minor things to change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Implements right-click and long-press. Currently not implemented in Gtk3 or WinUI. I'm also not sold on the argument label because
onTapGesture(gesture:) {}
looks a little odd.