-
-
Notifications
You must be signed in to change notification settings - Fork 243
-
Hello there!
I'm running Loop 1.4.2, and noticing some issues with URL commands:
- Running
loop://screen/nextorloop://screen/previousmoves the window, but resizes it to a very small size. - Attempting to run cycles (e.g.
loop://keybind/testcycle) doesn't work (cycle doesn't seem to execute) and also changes window size. - Running URL commands loads Loop's window.
Noting that none of these happen when using keybindings (which also feel quicker). If this would be possible, I think that URL commands should follow the exact same path as keybindings, so the behavior between them can't derive.
Relatedly, concerning instructions:
- I wonder if we should be proposing the use of the
open -gcommand (don't bring app to foreground) instead of plainopen, which seems sluggish on my system. I've triedopen -uas well in hopes that it would be faster, but it was slow. - FYI,
URLCommandHandler.swiftclaims that parameters with spaces must be URL encoded. In my quick non-extensive testing, this doesn't seem required, at least if we're using quotes (which all examples use). Just wanted to bring this up in case this information isn't valid anymore for any reason.
The way I'm running the commands is through karabiner, like this:
{
"description": "Testing Loop",
"manipulators": [
{ "description": "Test",
"from": {
"simultaneous": [{ "key_code": "slash" }, { "key_code": "t" }],
"simultaneous_options": { "key_down_order": "strict", "key_up_order": "strict_inverse" }
},
"parameters": { "basic.simultaneous_threshold_milliseconds": 500 },
"to": [{ "shell_command": "open -g 'loop://screen/previous'" }],
"type": "basic"
}
]
}
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 1 comment 2 replies
-
Running URL commands loads Loop's window.
I assume this should be fixed per #1076.
Beta Was this translation helpful? Give feedback.
All reactions
-
Thanks for bringing this up @acicovic! You are correct, #1076 will address all the issues you mentioned above, especially the speed issues, since you’ll be able to invoke Loop’s CLI directly instead of relying on URL calls for window resizing. This also removes the need to use open -g, for the same reason. The goal is to have this included in the next stable release :)
Beta Was this translation helpful? Give feedback.
All reactions
-
Thank you, looking forward to this! 🙂
Beta Was this translation helpful? Give feedback.