-
Notifications
You must be signed in to change notification settings - Fork 6
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
Allow buttons to work on monitors #7
Conversation
Now it's actually tested.
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.
Works fine and doesn't affect performance.
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 forgor to submit my review for this. Looks good, but I just need a few tiny tweaks to smoothen it out.
Actually now it requires the monitor argument, should I leave it like that or see if the window is a monitor and use that? |
True, I would prefer if you auto-detect monitors through |
I love github mobile bugs. |
Finally got around to doing it, also fixed a typo that broke everything (oops). |
I'm not actually sure how well this works, I'll have to test it more.
I've also made this in around 15 minutes, so it's not tested much.
Should this use the button component or be split into a different one?
Fixes #4 (i think)
Closes #5