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

Add matching title as the -t option #8

Closed
wants to merge 3 commits into from
Closed

Conversation

e3rd
Copy link
Contributor

@e3rd e3rd commented Sep 3, 2015

Use case:
I use Chromium feature to open Gmail webpage as an application window. Then I bind a hotkey to focus directly the Gmail window (and not other Chromium windows) or open it if it's closed:
jumpapp -t "Mail" chromium-browser --app=https://mail.google.com/mail/u/0

Then I bind another hotkey to cycle other Chromium windows or open Chromium if it's closed:
jumpapp -t Chromium chromium-browser --ssl-version-min=tls1

I use this feature a lot since months so I though you might want to make it accessible for everyone.

Use case:
I use Chromium feature to open Gmail webpage as an application window. Then I bind a hotkey to focus directly the Gmail window (and not other Chromium windows) or open it if it's closed:
jumpapp -t "Mail" chromium-browser --app=https://mail.google.com/mail/u/0

Then I bind another hotkey to cycle other Chromium windows or open Chromium if it's closed:
jumpapp -t Chromium chromium-browser --ssl-version-min=tls1


I use this feature a lot since months so I though you might want to make it accessible for everyone.
@mkropat
Copy link
Owner

mkropat commented Sep 3, 2015

Thanks for the PR! Matching on the title makes a lot of sense.

I won't be able to test your changes for a few days, but when I'm able I'll test it out and get this merged in.

mkropat added a commit that referenced this pull request Sep 26, 2015
mkropat added a commit that referenced this pull request Sep 26, 2015
@mkropat
Copy link
Owner

mkropat commented Sep 26, 2015

Merged to master with f60a0b6. I rebased your commits together, added unit tests, and refactored the code a bit. Thanks for you patience.

When you get a chance, can you pull down the latest master and make sure the title matching code still works for you?

@mkropat mkropat closed this Sep 26, 2015
@e3rd
Copy link
Contributor Author

e3rd commented Sep 28, 2015

Master's working, cool, thanks a lot ;)

@mkropat
Copy link
Owner

mkropat commented Sep 29, 2015

Perfect. Thanks for confirming.

@mkropat mkropat mentioned this pull request Dec 27, 2015
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