-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
File widget : Accept any file #6792
Comments
I like this suggestion now that the widget has an explicit file type selector. |
@markotoplak, I've started working on this and I have a few questions related to the current implementation.
The File widget's code is indeed one big mess. (And I confess to not only contributing to it but perhaps even establishing it in the first place.) It could be one's big project to reimplement if from scratch, or, better, based on (and merging it with) the CSV widget. |
It is stored as a part of the RecentPath object so it is stored in there.
I probably stored the reader into RecentPath because changing the file in the selector then also changed the reader. But I do agree with you: choosing a completely new file should in fact use the same reader as is currently selected.
I did not look into that part at all. In the end, the code uses the same readers but goes through different paths. Or maybe how I implemented the settings into a RecentPath was the problem... |
What's wrong?
I have files named
something.metar
but the file widget only accept certain file names.How can we reproduce the problem?
Open the widget and look to the drop down list of accepted file names : you won't find the
All files (*.*)
as usual.** NOTE **
Can somebody checks if the problem is Windows only, please.
What's your environment?
The text was updated successfully, but these errors were encountered: