UI improvements for hotkeys editing

[vlakoff]

Note I haven't tested these changes. A few points I can't say for sure:

• Maybe, MessageBox.Show would benefit from having a window as 1st parameter.
• Maybe, the dialog title updating would require a refresh.

You might also want to adjust the texts as you wish.

[xenolightning]

Providing the Window as the first parameter is preferable.

The Refresh after updating the title shouldn't be required. The text is updated before the initial render.

Suggestion for the text for the message boxes:

• "Do you want to clear all hotkeys?" - For Clear All
• "Do you want to delete this hotkey?" - For delete confirmation

If you want to make those changes I'll merge them in.

Thanks!

[vlakoff]

I have applied your suggestions.

I asked about the window as first argument because:

• the current "Hotkeys Cleared!" message doesn't have the argument but seems to work as expected
• in a few places in the codebase, the argument isn't used too

[xenolightning]

Yeah, there's not a huge amount of difference between the two. Generally wherever possible it should be use the base Window (Form), it just ensures the message box appears on top.

In Program.cs it's ok to leave them without, as we haven't created a Window yet.

I should probably change the UpdateForm one to use the base form too.

Will make that change, and most likely will update the underlying Api, since there's more recent versions now.