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

Blacklisting Alfred does not work #19

Closed
chrisgrieser opened this issue Sep 7, 2022 · 6 comments
Closed

Blacklisting Alfred does not work #19

chrisgrieser opened this issue Sep 7, 2022 · 6 comments

Comments

@chrisgrieser
Copy link

Inside the Alfred prompt, I don't really need vim, and I want to quit the prompt with esc. However, with svim, it takes 2-3 times escape before it quits.

Annoyingly, adding any of these to the blacklist does not work (and I confirmed that the blacklist does work with other apps)

"Alfred 5"
Alfred 5
Alfred
com.runningwithcrayons.Alfred

I also tried both, the Standard Mode and the Compatibility Mode of Alfred, blacklisting Alfred does not work in any of those
Pasted image 2022-09-07 22 47 30

@FelixKratz
Copy link
Owner

I think you would want to check in this function

static inline bool ax_get_selected_element(struct ax* ax) {

wether the host of the selected element is blacklisted. The current black listing does not work with alfred, since alfred never signals a front application changed event here
- (void)appSwitched:(NSNotification *)notification {

@FelixKratz
Copy link
Owner

I just tried it and using Alfred in compatibility mode with Alfred in the blacklist works here. Remember to restart svim after changing the blacklist.

@chrisgrieser
Copy link
Author

chrisgrieser commented Sep 8, 2022

I just tried it and using Alfred in compatibility mode with Alfred in the blacklist works here. Remember to restart svim after changing the blacklist.

I have done exactly that, added Alfred to the blacklist, enabled compatibility mode, and restarted svim via brew services restart svim, still not working.

@FelixKratz
Copy link
Owner

Could you try with the current master build? I am using that rather than the homebrew release. If this fixes the problem I will make a new release.

@FelixKratz FelixKratz reopened this Sep 9, 2022
@chrisgrieser
Copy link
Author

Can confirm, with the master it works. 🥳

@FelixKratz
Copy link
Owner

Great, new release is out now.

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

No branches or pull requests

2 participants