Hi,
We'll consider it, but I am not sure I like the solution. The main issue here is the "Fonts" folder hardcoded in the solution: It is a little bit better than just trying c:\Windows\Fonts directly. Yes, in a standard windows install fonts are in that folder, but you might have changed the defaults.
And for XP, SHGetFontFolder works fine, so it is a better option than GetEnvironmentVariable('SystemRoot') + '\Fonts'
So there is the chance that by doing this change we break an existing working FlexCel app which was used in XP and in a non-standard Fonts folder not located under the normal "Windows" folder. Whenever we make a change like this, we want to be as sure as possible we aren't breaking someone else's app.
On the other side, both XP and Win 2003 are out of support by Microsoft, so I am not sure on how much this matters.
But the supported solution (which is sure not to break any existing app) is to just use the OnGetFontFolder and write the code you wrote above in that event. That's why the OnGetFontFolder exists at all: to allow you to modify the standard GetFontPath result without needing to change the source code.
But I'll see if something can be added directly in our code. Maybe not checking for the os version (which would make it not work in XP) but trying to do SHGetFolderPath and only trying the Systemroot\Fonts alternative if SHGetFolderPath fails.