Skip to content
This repository has been archived by the owner on Mar 4, 2022. It is now read-only.

Make stream text mouse-selectable #30

Closed
wants to merge 1 commit into from

Conversation

gitnik
Copy link

@gitnik gitnik commented Oct 17, 2016

No description provided.

@jasonwilson
Copy link
Contributor

Thanks for the PR @gitnik. This doesn't seem to fix #26 for me-- what terminal and settings are you using?

@wmertens
Copy link

When using Terminal.app on OS X Mavericks, this does work.

@wmertens
Copy link

BTW, the graphs get included in the copy/paste. Perhaps they should be in a horizontal section of the screen?
e.g.:

Time: 9208ms                                                                                                                               ││     ⡇                                       │
│                               Asset     Size  Chunks             Chunk Names                                                              ││     ⡇                                       │
│                           broker.js  19.7 kB       4  [emitted]  broker                                                                   ││  50 ⡇                                       │
│24bcfa5936332a7f0742f5c61c2f69b6.jpg  7.71 kB          [emitted]                                                                           ││     ⡇                                       │
│                           server.js  8.61 MB       1  [emitted]  server                                                                   ││     ⡇                                       │
│                           worker.js   8.6 MB       2  [emitted]  worker                                                                   ││     ⡇                                       │
│                             init.js    20 kB       3  [emitted]  init                                                                     ││     ⡇                                       │
│           0_18d892acb8fe558a88ab.js  40.7 kB       0  [emitted]                                                                           ││  25 ⡇                                       │
│       0_18d892acb8fe558a88ab.js.map  43.9 kB       0  [emitted]                                                                           ││     ⡇                                       │
│                       server.js.map   8.4 MB       1  [emitted]  server                                                                   ││     ⡇                                       │
│                       worker.js.map   8.4 MB       2  [emitted]  worker                                                                   ││     ⡇                                       │
│                         init.js.map  21.3 kB       3  [emitted]  init                                                                     ││     ⡇                                       │
│                       broker.js.map  21.1 kB       4  [emitted]  broker                                                                   ││  0  ⡇⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠒⠒⠒⠒⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠤⠄      │
│ [415] multi server 40 bytes  [built]                                                                                                      ││     ⠓⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒⠒    │
│ [416] multi worker 64 bytes  [built]                                                                                                      ││      9  8   7  6   5   4  3   2  1   0      │
│    + 420 hidden modules                                                                                                                   ││                                             │

@wmertens
Copy link

a really annoying problem is that with this PR the scrolling of the boxes no longer works.

@@ -17,7 +17,7 @@ class StreamView {
inverse: true
},
keys: true,
mouse: true,
mouse: false,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

brilliant... finally. +1 I can now select stdout/err 👍

@wmertens
Copy link

Would it be possible to turn this on/off with a key? Not having scrollback is very limiting…

@wearhere
Copy link

Here's how to re-enable scrolling while also keeping text selection enabled: chjj/blessed#263 (comment)

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

Successfully merging this pull request may close these issues.

None yet

5 participants