(0007293)
Sebastien Barre (manager)
2007-04-13 12:49

Done.

(0007296)
Nicole Aucoin (reporter)
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 (reporter)
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 (manager)
2007-04-13 16:41

And "by doesn't work" you mean?

(0007305)
Sebastien Barre (manager)
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 (reporter)
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 (manager)
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 (reporter)
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 (manager)
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 (manager)
2007-04-13 17:35

OK, you should be all set now. *.blah should be handled.

(0007311)
Nicole Aucoin (reporter)
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!

Issue History
Date Modified	Username	Field	Change
2008-02-07 10:52	Sebastien Barre	Assigned To	Sebastien Barre => Yumin Yuan

Notes
(0007293) Sebastien Barre (manager) 2007-04-13 12:49	Done.

(0007296) Nicole Aucoin (reporter) 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 (reporter) 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 (manager) 2007-04-13 16:41	And "by doesn't work" you mean?

(0007305) Sebastien Barre (manager) 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 (reporter) 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 (manager) 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 (reporter) 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 (manager) 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 (manager) 2007-04-13 17:35	OK, you should be all set now. *.blah should be handled.

(0007311) Nicole Aucoin (reporter) 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!

Relationships