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

More sensible TerminalFinderButton #17

Closed
wants to merge 2 commits into from
Closed

Conversation

ashfinal
Copy link

Reuse current tab of current window instead of always opening new window;
You can change PWD by selecting certain folder as well, .app, .Workflow files are also supported;
Determine busy terminal window (vim, wget, etc) to open new window;
Use persist storage to show first-time launching greeting, update notification, and other staff.

@lexrus
Copy link
Owner

lexrus commented Jul 15, 2017

Awesome refinements. But I'm not familiar with Apple Script.

@ashfinal
Copy link
Author

ashfinal commented Jul 15, 2017

@lexrus I wrote some AppleScript before, but never really dug into it. After doing some Google search and write-run-modify loop, the shinny TerminalButton was out!
I've used this button for a few days, it worked as intended, as far as I know.
Also, the script was well documented, if you want to check it out or improve it. :)

@lexrus
Copy link
Owner

lexrus commented Jul 15, 2017

@ashfinal I respect what you did. But I prefer writing scripts in JavaScript for some reasons. Please keep your fork, I'll add a link in my README.md.

@lexrus lexrus closed this Jul 15, 2017
@ashfinal
Copy link
Author

OK, I would keep the fork.
I just realize that this script may be a little long (and grisly), which could be less short.

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.

2 participants