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

Prevent the copy event from bubbling up to body #76

Closed

Conversation

julienw
Copy link
Collaborator

@julienw julienw commented Apr 4, 2019

Fixes #75

@julienw
Copy link
Collaborator Author

julienw commented Apr 4, 2019

I didn't know how to run the tests (npm test runs Firefox but then nothing happens on my computer) so I didn't add any. But I'd be happy to with some guidance.

@sudodoki
Copy link
Owner

sudodoki commented Apr 4, 2019

Thanks! Pulled via ad759b0

@sudodoki sudodoki closed this Apr 4, 2019
julienw added a commit to julienw/perf.html that referenced this pull request Apr 4, 2019
@sudodoki
Copy link
Owner

sudodoki commented Apr 4, 2019

Wonder whether #46 (comment) workaround will work after this (although it performs copying by that code snippet already) 🤔

@julienw
Copy link
Collaborator Author

julienw commented Apr 4, 2019

ah yes indeed :/

@julienw
Copy link
Collaborator Author

julienw commented Apr 4, 2019

Maybe we can add this workaround right in the copy event handler, what do you think?
Unless there's another way to fix this...

@julienw
Copy link
Collaborator Author

julienw commented Apr 4, 2019

Another possibility is to set a white background to the span. Then the text would have a white background instead of a grey background. Still not perfect but better.

@julienw
Copy link
Collaborator Author

julienw commented Apr 4, 2019

Actually their workaround doesn't seem to work at all anyway. I'd suggest to just put a background to white.

@julienw
Copy link
Collaborator Author

julienw commented Apr 4, 2019

Ah yes it works, I had a bad state in my gdocs. Let me do a quick PR :)

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

Successfully merging this pull request may close these issues.

None yet

2 participants