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 lowfi command #970

Merged
merged 4 commits into from
Dec 11, 2024
Merged

add lowfi command #970

merged 4 commits into from
Dec 11, 2024

Conversation

nohzafk
Copy link
Contributor

@nohzafk nohzafk commented Nov 24, 2024

Description

Script command for start lowfi command in a wezterm window, run the second time to bring the lowfi window to the front

Type of change

  • New script command

Screenshot

Untitled

Dependencies / Requirements

Require

Checklist

@giedreblekaite
Copy link
Contributor

Thank you for your PR! Unfortunately, the script command is not working for me. I get this:
Screenshot 2024-12-03 at 16 59 15

@nohzafk
Copy link
Contributor Author

nohzafk commented Dec 3, 2024

Thank you for your PR! Unfortunately, the script command is not working for me. I get this:

Screenshot 2024-12-03 at 16 59 15

did you install lowfi ?

If you did, can you verify it's installed in one of the directories in your PATH, or add its location to your PATH

@giedreblekaite
Copy link
Contributor

Thank you for your PR! Unfortunately, the script command is not working for me. I get this:
Screenshot 2024-12-03 at 16 59 15

did you install lowfi ?

If you did, can you verify it's installed in one of the directories in your PATH, or add its location to your PATH

No, I didn't have it. Please update the comments that lowfi needs to be installed as well because currently they tell about wezterm only. By the way, I noticed that the script always starts a new lowfi instance even when wezterm window is visible. Maybe this could be fixed as well?

@nohzafk
Copy link
Contributor Author

nohzafk commented Dec 6, 2024

Thx for your feedback 😄 I've updated the instructions of how to install lowfi.

Could you install lowfi and then test it to see if the script can invoke the wezterm window if run the second time?

It's a little bitt tricky to get this effect work, I've spent quite some time on it.

Which macOS version are you using? I've tested it on the latest 15.1.1

@giedreblekaite
Copy link
Contributor

Thx for your feedback 😄 I've updated the instructions of how to install lowfi.

Could you install lowfi and then test it to see if the script can invoke the wezterm window if run the second time?

It's a little bitt tricky to get this effect work, I've spent quite some time on it.

Which macOS version are you using? I've tested it on the latest 15.1.1

Thanks for updated instructions. I've been using your script several times already on 15.1.1 but it always opens a new window instead of activating the current one. Could it be due to the fact that I have tabs set up like that? Maybe your window setup is different in any way.
Screenshot 2024-12-09 at 10 58 21

@nohzafk
Copy link
Contributor Author

nohzafk commented Dec 9, 2024

can you show your wezterm window that runs lowfi?

the apple script is trying to find a wezterm window whose title contains "lowfi" and then bring it to to the upfront and focus on it.

@giedreblekaite
Copy link
Contributor

can you show your wezterm window that runs lowfi?

the apple script is trying to find a wezterm window whose title contains "lowfi" and then bring it to to the upfront and focus on it.

Here's my screencast:

lowfi.mov

@nohzafk
Copy link
Contributor Author

nohzafk commented Dec 11, 2024

Thanks for you feedback and patience. I fix the issue, wezterm should run in the background when lowfi process starts.

The script worked previously if there is a wezterm window opened already, I tested both scenarios (no wezterm window exists, already a wezterm window exists ) and it works now.

@giedreblekaite
Copy link
Contributor

Thanks for you feedback and patience. I fix the issue, wezterm should run in the background when lowfi process starts.

The script worked previously if there is a wezterm window opened already, I tested both scenarios (no wezterm window exists, already a wezterm window exists ) and it works now.

Works great now, doesn't start a new instance if one is already playing. Good stuff! Thank you very much!

@giedreblekaite giedreblekaite merged commit 0c68505 into raycast:master Dec 11, 2024
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