Handle ignore unignore URL actions

[bric3]

Adds the support for

• open -g "rectangle://execute-action?name=ignore-app"
• open -g "rectangle://execute-action?name=unignore-app"

[rxhanson]

Thanks for looking to contribute.

Rather than adding these as window actions, I recommend adding a new url query parameter that will call applicationToggle.enableFrontApp() or applicationToggle.disableFrontApp(). That way your changes are only in AppDelegate. The url stuff is at the bottom of AppDelegate.

Maybe something like rectangle://execute-task?name=ignore-app?

[bric3]

What about adding a query pram for the app bundle id, so that even if IntelliJ is in background for some reason it can execute this action without affecting another app.

[rxhanson]

Sure! This looks good to me, but I can hold off on merging if you wanted to add in the bundleId query param.

[bric3]

OK I'll do it shortly.

[bric3]

Added app-bundle-id param for both ignore/unignore.

[rxhanson]

Thanks for making this change here. Sorry to beleaguer this, but I figure we might as well get this tidied up.

• What we have now can also be rewritten as return appBundleId ?? ApplicationToggle.frontAppId (hence one of the reasons why you don't typically see if come after return)
• However, this would still be the same as return appBundleId since once we get to ApplicationToggle, we would be providing the ApplicationToggle.frontAppId at that time in the case of appBundleId being nil.
• If we want to account for app-bundle-id= in the URL, then we would have to check that appBundleId != nil && appBundleId.isEmpty (depending on if we do indeed get back an empty string in that case - I didn't test it).

I'm still ok either way on doing the empty string check, but it still makes the most sense to me to at the very least change this to return appBundleId here.

[bric3]

Sorry, I pushed a change in the train yesterday as a reflex, (if anything happen to the laptop).

I agree with your points; my original goal is indeed the third point.