search

dekuNukem / duckyPad

Do-It-All Mechanical Macropad
MIT License
1.22k stars 169 forks source link

Closing duckypad config while saving causes partial shortcut loss #94

Closed fechu closed 2 years ago

fechu commented 2 years ago

Hi all,

I just had the case where I was saving scripts to the duckypad via duckypad config. I thought I had moved my focus to the browser where I wanted to close the current tab with CMD+W. That closed duckypad config. After opening again and loading the config part of the scripts are missing now.

Suggested solution: Show confirmation dialog when closing duckypad config if a saving operation is running.

@dekuNukem If you think this is a good improvement I'm happy to dig into the code and implement it. What do you think?

null0byte commented 2 years ago

+1 up vote for this fix. I closed the Config app forgetting it was saving and lost some minor changes. No massive issue but good UI fix if implemented.

dekuNukem commented 2 years ago

added in the latest update