Skip to content
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

firefox does not emit clipboard events #173

Closed
totaam opened this issue Nov 14, 2017 · 2 comments
Closed

firefox does not emit clipboard events #173

totaam opened this issue Nov 14, 2017 · 2 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Nov 14, 2017

Splitting this from #74 (comment:15)
Reproducible with Firefox on any platform.

This sounds like Capturing a Paste Event in Firefox: The paste event has been supported in major browsers for some time now, but interestingly Firefox doesn’t trigger the event unless the user’s cursor is currently focussed in an editable region.

Unfortunately, the Firefox bug is meant to have been fixed already and sprinkling contenteditable=true in every div doesn't make any difference. Neither does listening for clipboard events on all contenteditable elements as suggested here: [https://stackoverflow.com/a/19269040/428751].
More answers here: JavaScript get clipboard data on paste event (Cross browser)

@totaam
Copy link
Collaborator Author

totaam commented Nov 14, 2017

2017-11-14 16:17:27: antoine uploaded file test.html (7.7 KiB)

page similar to the client, clipboard works here...

@totaam
Copy link
Collaborator Author

totaam commented Nov 28, 2017

r17537 fixes this by not using jquery for keyboard events, which somehow fixes the clipboard with Firefox (yes, it did take me a while to figure that out)

Maybe we should try to not use jquery quite so much: #172.

A few improvements were made along the way:

  • r17531: remove redundant code
  • r17530: better logging and redirection
  • r17528 + r17529: refactoring, move javascript functions to common location, split client intialization into more manageable functions
  • r17527 fix audio api warnings

@totaam totaam closed this as completed Apr 22, 2018
@totaam totaam transferred this issue from Xpra-org/xpra May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant