Skip to content

Added file filter for drag & drop functionality.#29

Merged
pH-7 merged 1 commit into
pH-7:masterfrom
SidaDan:master
Oct 31, 2018
Merged

Added file filter for drag & drop functionality.#29
pH-7 merged 1 commit into
pH-7:masterfrom
SidaDan:master

Conversation

@SidaDan
Copy link
Copy Markdown
Contributor

@SidaDan SidaDan commented Oct 21, 2018

Hi again,
I added a file filter for drag & drop functionality and fixed a little bug
Bug fixed: If modified then ask to save the changes before drop a file.

Have a nice day.

Bug fixed: If modified then ask to save the changes before drop a file.
@pH-7
Copy link
Copy Markdown
Owner

pH-7 commented Oct 22, 2018

Looks amazing 💯 @SidaDan!!

fis.close();
}
catch (Exception ex) {
String[] extensionFilter = {".txt", ".dat", ".log", ".xml", ".mf", ".html"}; // allowed file filter extentions for drag and drop
Copy link
Copy Markdown
Owner

@pH-7 pH-7 Oct 22, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this array be a class attribute instead? We might have to reuse it later on elsewhere in the class... (and easier to find it if we have to add more file extensions). What do you think?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi. yes of course. Is only an example. You can create a setup.class as well.

Have a nice day

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that was a suggestion 😉

@pH-7 pH-7 merged commit 055b388 into pH-7:master Oct 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants