Hotkeys not working

[sinbad]

Hello

I have a long script and at the very end I have

Code: Select all

^r::Reload

But it isn't working

Why can that be?

[Scr1pter]

Hi,

Difficult to answer due to little information about your script.
Some questions/ideas:

1)
So this means nothing happens when you press Ctrl+R, right?

2)
If you replace reload by ExitApp, is it still the same?

3)
Do you work with #if blocks?

4)
Is any loop running?

Cheers!

[gregster]

Best guess: Add a little sleep to your loops.

[Hellbent]

Could be as simple as your script is sending keystrokes and thus you just need to add a * to your reload hotkey so that it fires regardless of any other keys being pressed.

But you did say that the reload hotkey wasn't working and not that it sometimes works and other times it doesn't so this is probably not the reason.
Just a shot in the dark.

Code: Select all

*^r::Reload

[sinbad]

Thanks @Hellbent, @gregster and @scr1pter I will work through your suggestions later today.

[sinbad]

The culprit was in some code I borrowed from github, it has

Code: Select all

SetTimer, EnableButtonSetup, 1

This is constantly checking the content of Gui edit boxes and enabling or disabling buttons. There must be a better way to do this?

[sinbad]

I may have spoken too soon. Having removed the SetTimer, my script still does not react to hotkeys. I tried the following and none had any affect.
^r::Reload
esc::ExitApp
F1::Reload

So I will keep looking...

[sinbad]

I think this is the #if is the problem. If I want to rewrite this inside of a function how could I do that? I keep getting an error that this is not allowed inside functions....

Code: Select all

#if WinActive(gui_title)
        ^c::
        send {AppsKey}c
        GuiControl, 1:, AuthCode, %clpbrd%

        If pwb !=
          ObjRelease(WEB)
        Gui, 2:destroy
        GuiControl, 1:+ReadOnly, ClientId
        GuiControl, 1:+ReadOnly, ClientSecret
    return