[Gtkradiant] Planning more Radiant file dialog fixes

Nerius Landys nlandys at gmail.com
Sat Dec 18 22:40:57 CST 2010


> Hi guys.  I'm going to fix at least 3 more bugs related to file
> dialogs in Radiant (Rambetter-temp-fixes branch) before I move on to
> anything else.
>
> First, the problem that a map isn't saved unless its filename ends in
> ".map" or ".xmap" (a file of 0 length is created but there is hardly
> any warning that your work hasn't been saved).  Reports regarding this
> problem have appeared several times, for example:
> http://zerowing.idsoftware.com/pipermail/gtkradiant/2010-February/011302.html
> .  This happens on Linux and on Windows.  Not sure yet how I'll fix it
> but likely something along the lines of a popup window that warn the
> user of the mistake, after the save operation fails.
>
> Number two on my list.  When using the GTK file dialog, files are not
> filtered in the file display.  So, if you're loading a map, you see,
> in the file dialog, all that crap in your maps folder, not only .map
> and .xmap files but also .bsp, .prt, .bak, and all the others.  It
> makes it really difficult to find what you're looking for.  Markus
> already fixed this bug in his GIT, and I will compare his change with
> whatever I come up with (and/or use his patch as a guide).
>
> Number three.  And I'm wondering if anyone has any thoughts on this.
> It's a Windows problem when using the native Windows file dialog
> (non-GTK option).  The problem is best described with a screenshot:
> http://daffy.nerius.com/temp/rad-file-dialog-no-refresh.png .  The
> thread in GtkRadiant that starts in function file_dialog() (in
> gtkmisc.cpp) passes control to the Windows function GetSaveFileName().
>  GetSaveFileName() does not return until the user has taken action
> (such as cancel or select file).  In the meantime, GtkRadiant cannot
> refresh itself because that thread is tied up.  Now is there an
> elegant solution to this problem?  I wonder how the GTK file dialog
> handles this.
>

Hello again.  All three issues are now fixed in Rambetter-temp-fixes
branch, waiting to be merged into trunk (as soon as that stabilizes).
In my opinion the file dialogs work really nice now.



More information about the Gtkradiant mailing list