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

Allow copying build logs easily #75

Closed
yuvipanda opened this issue Jul 17, 2017 · 8 comments
Closed

Allow copying build logs easily #75

yuvipanda opened this issue Jul 17, 2017 · 8 comments
Milestone

Comments

@yuvipanda
Copy link
Collaborator

The terminal can be fickle to copy build logs from. We should allow people to download them.

@choldgraf
Copy link
Member

@yuvipanda do the build logs still exist on the page even though they're no longer visible within the xterm window? Now that we're using the web clipper package could we just create a "copy logs" button?

@yuvipanda
Copy link
Collaborator Author

yuvipanda commented Jan 12, 2018 via email

@choldgraf
Copy link
Member

Would another option be to do this outside of xterm entirely? Basically any time some new data is written we can append that to a hidden div or something, and then copy/paste just copies from that div?

@yuvipanda
Copy link
Collaborator Author

yuvipanda commented Jan 14, 2018 via email

@kannes
Copy link

kannes commented Aug 17, 2018

Something I noticed:

If I do not uncollapse the log viewer right after loading the URL (and instead let Binder do its thing and uncollapse the viewer itself at the end), neither Firefox nor Chromium let me select any text in it.

Firefox does not let me copy the text at all.

Chromium requires me to right-click -> "Copy". Neither Ctrl-C nor Linux' usual select-to-copy copying work.

@choldgraf
Copy link
Member

Arg - thanks for your input @kannes!! We agree this is a pretty frustrating piece of the Binder UX right now :-/ if we can store the text in an element somewhere on the page, it should be pretty easy to use clipboard.js to give a "copy" button, but we need to make sure not to accidentally crash something in the process

@manics
Copy link
Member

manics commented Sep 20, 2021

Done in #1335

@manics manics closed this as completed Sep 20, 2021
@kannes
Copy link

kannes commented Sep 22, 2021

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants