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

Export the text as ctrl + s doesn't exist in HyperTerminal #2693

Closed
2 tasks done
theohamdan opened this issue Feb 21, 2018 · 20 comments
Closed
2 tasks done

Export the text as ctrl + s doesn't exist in HyperTerminal #2693

theohamdan opened this issue Feb 21, 2018 · 20 comments
Labels
🔌 Type: Plugin Request Issue is a plugin request for Hyper

Comments

@theohamdan
Copy link

theohamdan commented Feb 21, 2018

  • I am on the latest Hyper.app version
  • I have searched the issues of this repo and believe that this is not a duplicate
  • OS version and name: <macOS HighSierra 10.13.2>
  • Hyper.app version: < hyper 1.4.8 >
  • Link of a Gist with the contents of your .hyper.js:

Issue
Hi guys, when I open HyperTerminal I don't find "export the text as file" (like when I do ctrl + s) and it's very important as much in Terminal, then I want to know if anyone had already found this feature and I'm ready to participate your propositions

@ppot
Copy link
Contributor

ppot commented Feb 21, 2018

export text as ?

@lesce
Copy link
Contributor

lesce commented Feb 21, 2018

as a file , iterm2 and Terminal.app has this feat
first I ever heard of it too

@theohamdan
Copy link
Author

@ppot sorry, yes it's as file

@theohamdan
Copy link
Author

@lesce thanks for your participation

Then do you have any idea about the solution?

@lesce
Copy link
Contributor

lesce commented Feb 22, 2018

@mhamdan95 I can take a look tomorrow ,
I'm not very familiar with the codebase or electron apps for that matter , just started this week

@chabou chabou added the 🔌 Type: Plugin Request Issue is a plugin request for Hyper label Feb 23, 2018
@chabou
Copy link
Contributor

chabou commented Feb 23, 2018

Perfect idea for a plugin.
Saving a selection or all history 😍

@theohamdan
Copy link
Author

@chabou that's true, thanks a lot 👍

@theohamdan
Copy link
Author

hey guys, I didn't fix the feature yet, but I developed my ideas, and finally I found the root of the feature, it's in the folder app -> renderer in the file bundle.js

We have to add some operation to make work the "Export the text as"
Need your help 😟

@theohamdan
Copy link
Author

@chabou @lesce @ppot
Hi guys, do you have any proposition please?

@neil-orans
Copy link

@NicholasMorrison and I will be working on this issue in the coming weeks.

@theohamdan
Copy link
Author

yeaaah @neil-orans it will be just amazing, we can exchange our ideas together ;)

And I added the "save as" in the program but without the functionality. Then actually we have to do the function that makes the "save as" run.

@mrrsnn
Copy link

mrrsnn commented Apr 5, 2018

@mhamdan95 We have written and confirmed saving text that exists in the hyper terminal window. Just need to test it on multiple operating systems. The plugin will be released soon.

@chabou
Copy link
Contributor

chabou commented Apr 5, 2018

Be sure to target our canary version (our upcoming v2).

@mrrsnn
Copy link

mrrsnn commented Apr 5, 2018

@chabou If we simply pull from the canary branch and test our plugin with that version, would this suffice? Or is the canary version subject to a lot more change? The plugin mainly relies on being able to access the "term" object that is passed down to the _onTerminal function in most React Components that are implemented as hyper plugins. If the organization of the contents of the term object could possibly change a lot in the coming release of v2, then we will need to revise the plugin a bit.

@chabou
Copy link
Contributor

chabou commented Apr 5, 2018

It will be enough to test upon current canary (This should be our last canary before shipping our v2).
You'll still have access to Terminal instance (thanks to onTerminal hook), but be careful: xtermjs is now used instead of hterm as underlying terminal lib.

@neil-orans
Copy link

neil-orans commented Apr 6, 2018

@chabou the onTerminal hook doesn't seem to be working/executing if we use the script in the canary. Is this a known issue?

@chabou
Copy link
Contributor

chabou commented Apr 7, 2018

Yes, onTerminal hook doesn't work anymore. I don't know if we need to restore it because it seems quite too specific (and plugin that used it, would have an xtermjs instance and not a hterm one).

If you want to access xtermjs instance, you should use onDecorated hook on your Term decorator to get Term component instance and access to its term property.

@neil-orans
Copy link

The plugin is stable now on 1.4.8 and 2.0.0 (if you are using macOS) and can be found on npm or GitHub. Certain features are not fully working on Linux, and we haven't gotten around to testing Windows yet.

Feel free to give it a whirl and report any bugs or suggestions on our issues tracker!

@chabou
Copy link
Contributor

chabou commented Apr 16, 2018

@neil-orans Thank you so much for this plugin.
Do not hesitate to submit it to our store: https://github.com/zeit/hyper-site/wiki/Submitting-a-new-plugin-or-theme-to-Hyper-Store

@chabou chabou closed this as completed Apr 16, 2018
@neil-orans
Copy link

No problem! Credit also goes to @NicholasMorrison since much of this code was produced via pair-programming. Once we can do a little more testing, we will release and publish the 1.0.0 to npm and submit it to the hyper-store. Thanks for your help along the way @chabou!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔌 Type: Plugin Request Issue is a plugin request for Hyper
Projects
None yet
Development

No branches or pull requests

6 participants