Skip to content
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

Changes to the project to allow for Filter directory selection #50

Closed
wants to merge 2 commits into from

Conversation

pedroflima
Copy link

Pull request to address issue #1
A configuration for filter folder is now available under the File menu.

@jody jody linked an issue Oct 28, 2020 that may be closed by this pull request
4 tasks
@jody jody self-requested a review October 28, 2020 23:40
Copy link
Contributor

@jody jody left a comment

Choose a reason for hiding this comment

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

In testing, I can see the new menu item in the File menu. However, I am unable to select a folder from the popup chooser. (Also, this placement of the menu item is not intuitive.)

JVM: 11.0.6 (AdoptOpenJDK 11.0.6+10)
OS: Mac OS X 10.15.7 x86_64

@jody
Copy link
Contributor

jody commented Oct 29, 2020

The ability to select a directory is a configuration of JFileChooser.
See: https://docs.oracle.com/javase/tutorial/uiswing/components/filechooser.html

Not sure if that will address the problem I'm seeing, but it will improve the user experience.

@jody jody marked this pull request as draft November 4, 2020 22:27
@pedroflima
Copy link
Author

Dr @jody, the Pull Request has been amended with 1 more commit containing changes to use the JFileChooser, as last suggested. Testing on MacOS has not yet been possible (the team does not have boxes, physical or virtual, running MacOS)

Copy link
Contributor

@jody jody left a comment

Choose a reason for hiding this comment

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

There appear to be some undesired side-effects. Please check that this code isn't sending data to stdout and/or stderr inappropriately.

More importantly, I'm confused about the requirements for filters appearing in the alternative directory. In particular, what are the constraints on the package in which they appear? How are such filters intended to be compiled? These questions may indicate that the requirement (User Story #1) wasn't fully fleshed out and that the design wasn't adequately vetted, before this development proceeded.

I've just reviewed the associated issue (#1) and see that it is labeled as an epic and that the Acceptance Criteria checkbox isn't ticked. Both of those suggest that more initial work on specification are needed. My apologies for letting this PR go this long without realizing that there hadn't been adequate specification or design.

@pedroflima
Copy link
Author

Dr. @jody,

As we discussed, I will close this pull request and add information regarding the added complexity to issue #1

@jody jody closed this Nov 18, 2020
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.

[UserStory] UI change to allow filters in arbitrary directory
2 participants