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
[NETBEANS-4368] Fix test file navigation dialog #2155
Conversation
Fixes resizing of test file selection when navigating to test file. Files are sorted by path.
@junichi11 @tmysik Please review. |
Relevant NetBeans issue here. |
@@ -54,6 +55,14 @@ | |||
private static final Logger LOGGER = Logger.getLogger(GoToTest.class.getName()); | |||
private static final RequestProcessor RP = new RequestProcessor(GoToTest.class.getName(), 2); | |||
|
|||
private static class FileObjectComparator implements Comparator<FileObject> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note: I usually put the inner classes at the very end of the class (usually below //~ Inner classes
comment). No need to change it in this PR, I just think it makes the code more readable (main class and then inner classes, not mixed).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Thanks for the PR! |
@KacerCZ Please attach screenshots(before and after) if possible. Thanks for your contribution. |
@junichi11 I added screenshots to first comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
So, can we merge it? |
@tmysik No. NetBeans 12.0 has not been released yet. I'll let you know when we can do that :) |
@junichi11 OKOK, merging will be on you :) |
Fixes resizing of test file selection when navigating to test file.
Files are sorted by path.
Before:
After: