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

[ JDT LS ] The "Navigate File Form" has difference if compare with upstream #11490

Closed
Ohrimenko1988 opened this issue Oct 4, 2018 · 2 comments
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@Ohrimenko1988
Copy link
Contributor

Note
The bug is actual for this assembly:
#10863

Reproduction Steps

  • Run default "Java" stack with default "Spring" project
  • Open "GreetingController.java" file
  • Type in the class body
       private int privateVariable = 1;
       public int publicVariable = 1;
  • Press "Ctrl" + "F12"
  • In the opened form press "Ctrl" + "F12" once more
  • Pay attention to icons

Expected behavior:
docker

Observed behavior:
hhhhhheee

OS and version:
JDT LS assembly Che 6.12.0; Chrome 68; Fedora 28

@Ohrimenko1988 Ohrimenko1988 added kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system. team/languages labels Oct 4, 2018
@tsmaeder
Copy link
Contributor

tsmaeder commented Oct 8, 2018

Yes, the rendering and sorting is slightly different. I think the rendering is better, because you can't override based on return type, so that info is useless here.
Need to check why the icon is "private"

@tsmaeder
Copy link
Contributor

tsmaeder commented Oct 8, 2018

I can confirm that the visiblity is indeed ignored. This is actually a LSP problem: microsoft/language-server-protocol#98

We can fix this for the file structure window, though, since it uses a custom command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

2 participants