search

tilmanginzel / alfred-bluetooth-workflow

Yet another Alfred workflow to connect / disconnect Bluetooth devices
MIT License
189 stars 8 forks source link

Use blt-t command instead of blt-toggle #4

Closed tilmanginzel closed 5 years ago

tilmanginzel commented 5 years ago

@trietsch Please check if this is fine for you. @mlisitski has proposed to just use blt-t instead of blt-toggle, as its faster to type (#1). I think it's a good idea.

We could release a new version v0.4.0 once it is merged. I was also wondering: is it necessary to always provide the version file when releasing a new workflow? I think it then would make sense to check this file into the git repository too.

By the way, feel free to add yourself to the license.

trietsch commented 5 years ago

Hey @tilmanginzel, nice work on the PR :) Regarding blt-toggle vs blt-t, I personally prefer the full name, since you've got auto-complete when pressing enter/return anyways. Also, toggle indicates what it does, while t doesn't. But I don't have a strong opinion on it, so whatever you prefer.

trietsch commented 5 years ago

Also, it's always necessary to include a version file, with the version number as plain text. You can read more on it on Alfred Workflow. It would be good to add it to the repo.

tilmanginzel commented 5 years ago

Regarding blt-toggle vs blt-t, I personally prefer the full name, since you've got auto-complete when pressing enter/return anyways. Also, toggle indicates what it does, while t doesn't.

Hm I have given it a second thought and agree that we should stick with blt-toggle for now. blt-on/off are also not shortened. And you can always just use your mouse or a keyboard shortcut for a quick selection.