MantisBT - KWWidgets
View Issue Details
0004864KWWidgetspublic2007-04-13 12:022008-02-07 10:52
Nicole Aucoin 
Yumin Yuan 
lowfeaturealways
assignedfixed 
0004864: REQ: LoadSaveDialog SetFileTypes string formatting
SetFileTypes doesn't work (get no file types offered in the GUI) if you have a space between the curly braces that delimit a file type name/pattern pair, and the braces that delimit the name and the pattern.
For example, this doesn't work:
 "{ {All} {*.*} }"
while this does:
 "{{All} {*.*}}"
The API documentation shows an example with the second format, but it would increase readability if the first format string worked as well.
No tags attached.
Issue History
2008-02-07 10:52Sebastien BarreAssigned ToSebastien Barre => Yumin Yuan

Notes
(0007293)
Sebastien Barre   
2007-04-13 12:49   
Done.
(0007296)
Nicole Aucoin   
2007-04-13 13:13   
Both format strings work now, thanks!
(0007302)
   
2007-04-13 16:34   
Could you please add a SetFileTypes call in the widgets tour example? It doesn't look like when I set file types that it's actually filtering the files in the new file browser.
(0007303)
Nicole Aucoin   
2007-04-13 16:36   
The filtering on type doesn't work in the new file browser on linux 64.
Could you add a SetFileTypes call to the widgets tour example so it's easier to test independently?
(0007304)
Sebastien Barre   
2007-04-13 16:41   
And "by doesn't work" you mean?
(0007305)
Sebastien Barre   
2007-04-13 16:43   
I have it on front of me and it's working fine on Linux, I can see the different FileTypes in the KWFileBrowserDialogExample (which is also where you can easily modify the FileTypes).
(0007306)
Nicole Aucoin   
2007-04-13 16:57   
The list of files visible in the file listing doesn't change when I change the filter type in the drop down menu. I checked your example, that is working, and if I put spaces between the brackets and recompile it, it still works, so it's not the spaces in my SetFileTypes call. I did have the extensions specified as *.blah instead of .blah, using the latter, it started filtering properly again.
So, I think that the larger issue is that the Tcl/Tk format strings we have in Slicer3 aren't fully supported with the new file dialogues.
(0007307)
Sebastien Barre   
2007-04-13 17:07   
so what you are telling me it's that the *.blah doesn't work vs. .blah. It's a really big piece of code we are talking here that's why I'm asking, I need you to be precise about what's "not working" because we can't just look over the whole code at once :)
(0007308)
Nicole Aucoin   
2007-04-13 17:14   
Sorry, I was trying to work out what was wrong as I was typing, so I wasn't very clear.
So, feature request addendum: could you add support for *.blah as a way of specifying extensions in the SetFileTypes call, in addition to the already supported .blah?
Currently, when I use *.blah, the file list shows files matching *.*
(0007309)
Sebastien Barre   
2007-04-13 17:31   
Actually the file types is a list of extension. It was never meant to work with *.blah. You are telling me it used to work?
(0007310)
Sebastien Barre   
2007-04-13 17:35   
OK, you should be all set now. *.blah should be handled.
(0007311)
Nicole Aucoin   
2007-04-13 17:59   
Yes, it definitely used to work as I hadn't changed my format string and the dialogue box used to let me filter on my *.blah types. Double checking the TK documentation:
http://tcl.activestate.com/man/tcl8.4/TkCmd/getOpenFile.htm#M13 [^]
*.blah wasn't supposed to work, but thanks for adding it back in!