-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add easy way to change hterm preferences #60
Comments
Thanks, Michael, for volunteering for this. I hadn't seen Secure Shell's page for this. I wonder if we can just bring up the same page as they do, but showing Mosh assets. That'd be much easier from a maintenance standpoint, too. |
I just looked at how Secure Shell works, and unfortunately, the preferences editor is under the |
I haven't looked closely enough at the code to really understand how tightly coupled it is to I'll try to hack something up in the next week, if just to get a little more familiar with the chrome extension environment. |
@rginda how do you feel about moving some of the hterm preference stuff out of nassh and into hterm itself ? that way people using hterm have an existing options page they can just drop in and run with. |
I intentionally kept it out of hterm for a few reasons, but we could revisit it. My concerns were:
#1 and #2 could be addressed by showing the options in a div overlay on top of hterm itself. Or we could put some of the prefs in a context menu and avoid a prefs page completely. For #3, I think we'd hard-code English strings but provide a way for the embedding app to override them with locale specific translations. This is basically what we have with hterm.zoomWarningMessage/notifyCopyMessage, except we'd probably want to switch to a dictionary of messages. (edited for formatting/typos) |
Thank you for the advise |
Fixed in SHA c25e02f. |
It would be nice if there was a standard preferences page to make editing them easy, without doing
term_.prefs_.set('foo')
.This is largely selfishly motivated, and I volunteer to add it, but it might take a while to me to get to it. Secure Shell does something like this already.
The text was updated successfully, but these errors were encountered: