2015-12-14, 00:08
Ronie is a good person to link to - when it comes to Python my skill level is far closer to 'enthusiastic hacker' than 'skilled coder' - he has the understanding of the language and the api, as well as the knowledge to understand why this is an issue. He probably even knows how focusing on the relevant control can be delayed within the script until the shortcuts are actually loaded. (And as we haven't quite gone ahead with a Jarvis repo push of the script yet, there might just about still be time to get a fix in before such happens!)
Edit:- Two points
First, having had a quick look at the mailing list post, I'd suggest this has only come to light because of the various undefined ID's that Ronie found. I believe that most every skin using Skin Shortcuts is likely to have the same issue - and if it hasn't blocked them from being on the repo... (which isn't to say it's not something that shouldn't be fixed!)
Second, I haven't reviewed the code used when loading the window for a long long time, and my understanding of Python has improved since then. From what I understand, the loading of the user shortcuts and the displaying of them should probably be in the __init__ function, rather than the onInit function where it currently is. I'm fast running out of time tonight - which, with my schedule for the next couple of days means I probably can't change + test it in time for inclusion in the next repo release - but this should be fixable for the repo+1 release.
Edit:- Two points
First, having had a quick look at the mailing list post, I'd suggest this has only come to light because of the various undefined ID's that Ronie found. I believe that most every skin using Skin Shortcuts is likely to have the same issue - and if it hasn't blocked them from being on the repo... (which isn't to say it's not something that shouldn't be fixed!)
Second, I haven't reviewed the code used when loading the window for a long long time, and my understanding of Python has improved since then. From what I understand, the loading of the user shortcuts and the displaying of them should probably be in the __init__ function, rather than the onInit function where it currently is. I'm fast running out of time tonight - which, with my schedule for the next couple of days means I probably can't change + test it in time for inclusion in the next repo release - but this should be fixable for the repo+1 release.