-
Notifications
You must be signed in to change notification settings - Fork 27
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
use message to open duckplayer from overlay #992
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @shakyShane and the rest of your teammates on |
2a4fd4e
to
040aec4
Compare
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.
on macOS tested:
- with overlay, tap on watch in duck player
- with overlay, tap on watch on youtube
- without overlay, opening directly
working as expected, with the difference that the overlay correctly calls the handler now
040aec4
to
e724efc
Compare
56832ff
to
886ef91
Compare
bdf08a2
to
ac6f6c8
Compare
74238ee
to
8eef7fb
Compare
8eef7fb
to
d188692
Compare
Temporary Branch UpdateThe temporary branch has been updated with the latest changes. Below are the details:
Please use the above install command to update to the latest version. |
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.
👍
https://app.asana.com/0/1201141132935289/1207702853516473/f
This just removes a single instance where the page location was updated by the JS.
Note This needs testing on macOS + Windows