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

Make RK/RK win apps available as a priority app in Focus Assist #112

Open
willwade opened this issue Jul 19, 2022 · 8 comments
Open

Make RK/RK win apps available as a priority app in Focus Assist #112

willwade opened this issue Jul 19, 2022 · 8 comments

Comments

@willwade
Copy link
Contributor

Is your feature request related to a problem? Please describe.

For users to use AAC - which is typically fullscreen - they will only get notifications if they turn off a bunch of settings. Which is ugly. https://docs.acecentre.org.uk/products/v/relaykeys/guides#a-tip-on-notifications

If it was a "Priority" App it would be a bit nicer.

Describe the solution you'd like

Make RK/equivalents a Priority app. We should be able to do this using AppSetUserID - I'm trying this with a NSIS extension like seen in this PR #111 - but have no idea if its working. It doesn't look like it.

as @joedevsys says

"I compiled the notify-py simple example which works ok. Setting Focus Assist=Priority Only stops it popping up as you'd expect (the event is shown in the events list ok), however it's not possible to add the test program to the Focus Assist Priority List so the popup's can't be enabled again - the app would need to register an AppUserModelId before it can be added to the priority list. Not sure if this library supports this. https://github.com/DatGuy1/Windows-Toasts might be better but lacking documentation of this area so going to need some digging."

And then me:

"Thanks @joedevsys -I'll look into that AppUserModelId - it looks like its more of an installer thing rather than the actual notify thing (https://docs.microsoft.com/en-us/windows/win32/shell/enable-desktop-toast-with-appusermodelid). If thats the case we should be able to register cli-win.exe cli.exe with a AppUserModelId using this https://nsis.sourceforge.io/WinShell_plug-in - but still not 100% convinced.

By the way - a large amount of commits to the workflow later - you can grab the latest installer on any commits to master on the action tab. e.g. (see at the bottom under artifacts https://github.com/AceCentre/RelayKeys/actions/runs/2684972767 )"

See #105 (comment)

Describe alternatives you've considered

https://github.com/safing/nsis-shortcut-properties

Not worked either.. Well definitely app is getting registered correctly. But I dont see it in a priority list

Additional context

We could do a popup modal or similar - but it won't work as the AAC app has fullscreen view. It would just get hidden in the background.

@joedevsys
Copy link
Contributor

The toast notifications are neat apart from the difficulty with Focus Assist.

An alternative for AAC apps could be to have a --copy switch. Put the return message into the clipboard so the app can display in a cell. e.g. Communicator5 cell script would be something like this to switch put the new device name into the message bar:
Clear
Write "Switching device, please wait..."
Run Program: relaykeys-cli-win.exe cle_cmd:switch
Delay 8 secs
Run Program: relaykeys-cli-win.exe cle_cmd:devname --notify
Paste

It's a bit clunky so getting toast to register properly would be best.

willwade added a commit that referenced this issue Jul 24, 2022
@willwade
Copy link
Contributor Author

willwade commented Jul 24, 2022

I think trying it out isn't a bad idea - Funnily enough I thought of another similar idea..
I've added a --copy flag - which means you can do --notify &/or --copy. Would still like to get to the bottom of how you register this properly mind you

See #121

@joedevsys
Copy link
Contributor

Bug in your PR: Line 179 call to copy_return() 2nd parameter command is unknown, should be devcommand.
Suggestion added to the PR.

With this fix --copy flag works really well in Communicator5 using the script in my previous comment. This is an effective alternative, but the user & I both would prefer toast notifications if the Focus Assist can be resolved.

@willwade
Copy link
Contributor Author

ok - well im a fan of options.. lets merge it and still keep plugging away at that issue

@joedevsys
Copy link
Contributor

--copy working great after merging modified #121

@willwade
Copy link
Contributor Author

willwade commented Nov 9, 2022

This might help - doubt it - but might https://github.com/binwiederhier/ntfy

@willwade
Copy link
Contributor Author

Joe - I have a favour to ask. Can you install this app we pulled together. https://github.com/AceCentre/TranslateAndTTS#step-1-download

Dont worry - you dont really need to set it up. Just add a call in communicator to the exe.

I want to get it to create an alert. Just getting it offline should do it. Get communicator full screen.

Does the alert come up with UAC and all that set up? Im kind of incredulous as it is for me.. and I didnt think it would

If thats the case the way we do notifications (QT with a timeout). We might copy over to relaykeys.. but I don't believe it.

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