Skip to content

Support right click context for viewer nodes #208

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.

Sign up for GitHub

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

Merged
testforstephen merged 11 commits into from
Dec 26, 2019
Merged

Support right click context for viewer nodes #208

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
cb37fd6
Support URI for nodes created from classpath entry
Vigilans Nov 19, 2019
6198969
Support URI for PackageRootNode
Vigilans Nov 19, 2019
6cd2ad0
Support URI for PackageFragment
Vigilans Nov 19, 2019
d381c90
Support Right Click Context in Client Side
Vigilans Nov 19, 2019
3fbdb78
Change description for `revealFileInOS`
Vigilans Nov 20, 2019
f2f740b
Hide right-click-context commands from command palette
Vigilans Nov 20, 2019
cba229e
Merge remote-tracking branch 'origin/master' into vigilans/right-clic…
Vigilans Nov 26, 2019
c87df7d
Add context value for testing uri
Vigilans Nov 26, 2019
2949f08
Extract detecting container uri as util method
Vigilans Nov 26, 2019
3efc15b
Fix getContainerURI NPE
Vigilans Dec 9, 2019
9f71db3
Merge remote-tracking branch 'origin/master' into vigilans/right-clic…
Vigilans Dec 9, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 13 additions & 7 deletions jdtls.ext/com.microsoft.jdtls.ext.core/src/com/microsoft/jdtls/ext/core/PackageCommand.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,9 +175,9 @@ public static List<PackageNode> resolvePath(List<Object> arguments, IProgressMon
IPackageFragmentRoot pkgRoot = (IPackageFragmentRoot) packageFragment.getParent();
PackageNode rootNode = null;

rootNode = new PackageRootNode(
rootNode = new PackageRootNode(pkgRoot,
ExtUtils.removeProjectSegment(packageFragment.getJavaProject().getElementName(), pkgRoot.getPath()).toPortableString(),
pkgRoot.getPath().toPortableString(), NodeKind.PACKAGEROOT, pkgRoot.getKind());
NodeKind.PACKAGEROOT);

result.add(PackageNode.createNodeForProject(packageFragment));
result.add(rootNode);
Expand Down Expand Up @@ -209,8 +209,8 @@ private static List<PackageNode> getParentAncestorNodes(IResource element) throw
IJavaElement javaElement = JavaCore.create(element);
if (javaElement instanceof IPackageFragmentRoot) {
IPackageFragmentRoot pkgRoot = (IPackageFragmentRoot) javaElement;
nodeList.add(0, new PackageRootNode(element.getProjectRelativePath().toPortableString(), pkgRoot.getPath().toPortableString(),
NodeKind.PACKAGEROOT, pkgRoot.getKind()));
nodeList.add(0, new PackageRootNode(pkgRoot,
element.getProjectRelativePath().toPortableString(), NodeKind.PACKAGEROOT));
nodeList.add(0, PackageNode.createNodeForProject(javaElement));
return nodeList;
} else if (javaElement instanceof IPackageFragment) {
Expand Down Expand Up @@ -280,8 +280,7 @@ private static List<PackageNode> getPackageFragmentRoots(PackageParams query, IP
if (fragmentRoot.getKind() == IPackageFragmentRoot.K_SOURCE) {
displayName = ExtUtils.removeProjectSegment(javaProject.getElementName(), fragmentRoot.getPath()).toPortableString();
}
PackageRootNode node = new PackageRootNode(displayName, fragmentRoot.getPath().toPortableString(), NodeKind.PACKAGEROOT,
fragmentRoot.getKind());
PackageRootNode node = new PackageRootNode(fragmentRoot, displayName, NodeKind.PACKAGEROOT);
children.add(node);
if (fragmentRoot instanceof JrtPackageFragmentRoot) {
node.setModuleName(fragmentRoot.getModuleDescription().getElementName());
Expand Down Expand Up @@ -440,7 +439,14 @@ private static List<PackageNode> convertToPackageNode(Object[] rootContent, IPac
List<PackageNode> result = new ArrayList<>();
for (Object root : rootContent) {
if (root instanceof IPackageFragment) {
result.add(PackageNode.createNodeForPackageFragment((IPackageFragment) root));
IPackageFragment fragment = (IPackageFragment) root;
PackageNode entry = PackageNode.createNodeForPackageFragment(fragment);
if (fragment.getResource() != null) {
entry.setUri(fragment.getResource().getLocationURI().toString());
} else {
entry.setUri(fragment.getPath().toFile().toURI().toString());
}
result.add(entry);
} else if (root instanceof IClassFile) {
IClassFile classFile = (IClassFile) root;
PackageNode entry = new PackageNode(classFile.getElementName(), null, NodeKind.TYPEROOT);
Expand Down
38 changes: 26 additions & 12 deletions ....ext/com.microsoft.jdtls.ext.core/src/com/microsoft/jdtls/ext/core/model/PackageNode.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
import org.eclipse.jdt.core.IPackageFragmentRoot;
import org.eclipse.jdt.core.JavaCore;
import org.eclipse.jdt.core.JavaModelException;
import org.eclipse.jdt.launching.JavaRuntime;
import org.eclipse.jdt.ls.core.internal.JDTUtils;

import com.microsoft.jdtls.ext.core.ExtUtils;
Expand Down Expand Up @@ -133,11 +134,11 @@ public static PackageNode createNodeForPackageFragmentRoot(IPackageFragmentRoot
if (entry.getEntryKind() == IClasspathEntry.CPE_VARIABLE) {
return createNodeForClasspathVariable(entry);
} else {
return new PackageRootNode(pkgRoot.getElementName(), pkgRoot.getPath().toPortableString(), NodeKind.PACKAGEROOT, pkgRoot.getKind());
return new PackageRootNode(pkgRoot, pkgRoot.getElementName(), NodeKind.PACKAGEROOT);
}
} else {
return new PackageRootNode(ExtUtils.removeProjectSegment(pkgRoot.getJavaProject().getElementName(), pkgRoot.getPath()).toPortableString(),
pkgRoot.getPath().toPortableString(), NodeKind.PACKAGEROOT, pkgRoot.getKind());
return new PackageRootNode(pkgRoot,
ExtUtils.removeProjectSegment(pkgRoot.getJavaProject().getElementName(), pkgRoot.getPath()).toPortableString(), NodeKind.PACKAGEROOT);
}
}

Expand All @@ -161,17 +162,29 @@ public static PackageNode createNodeForClasspathEntry(IClasspathEntry classpathE
container = JavaCore.getClasspathContainer(entry.getPath(), javaProject);
}
if (container != null) {
switch (nodeKind) {
case CONTAINER:
return new ContainerNode(container.getDescription(), container.getPath().toPortableString(), nodeKind, entry.getEntryKind());
case PACKAGEROOT:
PackageNode node = null;
if (nodeKind == NodeKind.CONTAINER) {
node = new ContainerNode(container.getDescription(), container.getPath().toPortableString(), nodeKind, entry.getEntryKind());
switch (container.getKind()) {
Comment thread
Vigilans marked this conversation as resolved.
Outdated
case IClasspathContainer.K_DEFAULT_SYSTEM: // JRE Container
case IClasspathContainer.K_SYSTEM:
node.setUri(JavaRuntime.getVMInstall(javaProject).getInstallLocation().toURI().toString());
break;
case IClasspathContainer.K_APPLICATION: // Plugin Container, Maven Container, etc
break; // No good way to find out the root uri currently
default: // Persistent container (e.g. /src/main/java)
node.setUri(container.getPath().toFile().toURI().toString());
}
} else if (nodeKind == NodeKind.PACKAGEROOT) { // ClasspathEntry for referenced jar files
// Use package name as package root name
String[] pathSegments = container.getPath().segments();
return new PackageRootNode(pathSegments[pathSegments.length - 1], container.getPath().toPortableString(), nodeKind,
IPackageFragmentRoot.K_BINARY);
default:
return null;
node = new PackageRootNode(
pathSegments[pathSegments.length - 1],
container.getPath().toPortableString(),
container.getPath().toFile().toURI().toString(),
nodeKind, IPackageFragmentRoot.K_BINARY);
}
return node;
}
} catch (CoreException e) {
JdtlsExtActivator.logException("Problems when convert classpath entry to package node ", e);
Expand Down Expand Up @@ -201,7 +214,8 @@ public static PackageNode createNodeForClasspathVariable(IClasspathEntry classpa
IClasspathEntry entry = JavaCore.getResolvedClasspathEntry(classpathEntry);
String name = classpathEntry.getPath().toPortableString();
String path = entry.getPath().toPortableString();
return new PackageRootNode(name, path, NodeKind.PACKAGEROOT, IPackageFragmentRoot.K_BINARY);
String uri = entry.getPath().toFile().toURI().toString();
return new PackageRootNode(name, path, uri, NodeKind.PACKAGEROOT, IPackageFragmentRoot.K_BINARY);
}

public String getName() {
Expand Down
15 changes: 14 additions & 1 deletion .../com.microsoft.jdtls.ext.core/src/com/microsoft/jdtls/ext/core/model/PackageRootNode.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,17 +13,30 @@

import java.util.Map;

import org.eclipse.jdt.core.IPackageFragmentRoot;
import org.eclipse.jdt.core.JavaModelException;

public class PackageRootNode extends PackageNode {

private int entryKind;

private Map<String, String> attributes;

public PackageRootNode(String name, String path, NodeKind kind, int entryKind) {
public PackageRootNode(String name, String path, String uri, NodeKind kind, int entryKind) {
super(name, path, kind);
this.setUri(uri);
this.entryKind = entryKind;
}

public PackageRootNode(IPackageFragmentRoot pkgRoot, String name, NodeKind kind) throws JavaModelException {
this(name, pkgRoot.getPath().toPortableString(), null, kind, pkgRoot.getKind());
if (pkgRoot.getResource() != null) {
this.setUri(pkgRoot.getResource().getLocationURI().toString());
} else {
this.setUri(pkgRoot.getPath().toFile().toURI().toString());
}
}

public int getEntryType() {
return this.entryKind;
}
Expand Down
34 changes: 33 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"explorer"
],
"engines": {
"vscode": "^1.30.0"
"vscode": "^1.31.0"
},
"repository": {
"type": "git",
Expand Down Expand Up @@ -83,6 +83,21 @@
"dark": "images/dark/icon-link.svg",
"light": "images/light/icon-link.svg"
}
},
{
"command": "java.view.package.revealFileInOS",
Copy link
Copy Markdown
Contributor

@testforstephen testforstephen Nov 28, 2019

Choose a reason for hiding this comment

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

revealFileInOS -> revealInExplorer

Copy link
Copy Markdown
Member Author

@Vigilans Vigilans Nov 28, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

revealInExplorer is not appropriate since it is related to two built-in commands in VS Code: revealInExplorer and git.revealInExplorer - They all refer to the meaning of "Reveal the file node in VS Code File Explorer", not the OS explorer.

In contrast, revealFileInOS is a built-in command of VS Code to open a file in OS explorer, and our command just forwards the dependency node's uri to it, so this naming would be fine.

Copy link
Copy Markdown
Contributor

@testforstephen testforstephen Nov 28, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

so why not to mount the VS Code builtin command to the dependency explorer directly?

Copy link
Copy Markdown
Member Author

@Vigilans Vigilans Nov 28, 2019

Choose a reason for hiding this comment

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

It would be best that we could mount the builtin command directly.
But the builtin command accepts a Uri argument, where context entry specified in package.json only allows for passing a ExplorerNode to the command binding.

(The TreeItem.command property can indeed specify the argument to be passed, but this is not the case with view/item/context entry)

Copy link
Copy Markdown
Contributor

@testforstephen testforstephen Dec 2, 2019

Choose a reason for hiding this comment

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

It seems VS Code provides different names in different OSes, could you check whether we need keep consistent?

Copy link
Copy Markdown
Member Author

@Vigilans Vigilans Dec 9, 2019

Choose a reason for hiding this comment

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

Sorry for I didn't notice this comment...

As this file shows, VS Code does provide different names here:

  • Windows: Reveal In Explorer
  • Mac OS: Reveal In Finder
  • Linux: Open Containing Folder

And the Chinese translation uses that localization:

"vs/workbench/contrib/files/electron-browser/fileActions.contribution": {
    "revealInWindows": "在资源管理器中显示",
    "revealInMac": "在 Finder 中显示",
    "openContainer": "打开所在的文件夹",
    "filesCategory": "文件"
},

Currently I have no knowledge of how to utilize the localization here according to OS...

Copy link
Copy Markdown
Contributor

@testforstephen testforstephen Dec 26, 2019

Choose a reason for hiding this comment

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

Currently i'm OK to keep the same one, if there is user complaint, then fix it.
A workaround is add three difference commands, and use when clause to enable the correct one.

"title": "%contributes.commands.java.view.package.revealFileInOS%",
"category": "Java"
},
{
"command": "java.view.package.copyFilePath",
"title": "%contributes.commands.java.view.package.copyFilePath%",
"category": "Java"
},
{
"command": "java.view.package.copyRelativeFilePath",
"title": "%contributes.commands.java.view.package.copyRelativeFilePath%",
"category": "Java"
}
],
"configuration": {
Expand Down Expand Up @@ -147,6 +162,23 @@
"when": "view == javaDependencyExplorer && config.java.dependency.syncWithFolderExplorer == true",
"group": "navigation@0"
}
],
"view/item/context": [
{
"command": "java.view.package.revealFileInOS",
"when": "view == javaDependencyExplorer",
"group": "@1"
},
{
"command": "java.view.package.copyFilePath",
"when": "view == javaDependencyExplorer",
"group": "@2"
},
{
"command": "java.view.package.copyRelativeFilePath",
"when": "view == javaDependencyExplorer",
"group": "@2"
}
]
},
"views": {
Expand Down
3 changes: 3 additions & 0 deletions package.nls.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@
"contributes.commands.java.view.package.changeToHierarchicalPackageView":"Change to hierarchical package representation",
"contributes.commands.java.view.package.linkWithFolderExplorer":"Synchronize dependency viewer selection with folder explorer",
"contributes.commands.java.view.package.unlinkWithFolderExplorer":"Desynchronize dependency viewer selection with folder explorer",
"contributes.commands.java.view.package.revealFileInOS": "Open Containing Folder",
Comment thread
Vigilans marked this conversation as resolved.
Outdated
"contributes.commands.java.view.package.copyFilePath": "Copy Path",
"contributes.commands.java.view.package.copyRelativeFilePath": "Copy Relative Path",
"configuration.java.dependency.title": "Java Dependency Configuration",
"configuration.java.dependency.showOutline": "Enable show outline in the Java Dependency explorer",
"configuration.java.dependency.syncWithFolderExplorer": "Synchronize dependency viewer selection with folder explorer",
Expand Down
3 changes: 3 additions & 0 deletions package.nls.zh.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@
"contributes.commands.java.view.package.changeToHierarchicalPackageView":"将 Java 包显示方式切换为分层显示",
"contributes.commands.java.view.package.linkWithFolderExplorer":"开启 Java 依赖项资源管理器与当前浏览文件的关联",
"contributes.commands.java.view.package.unlinkWithFolderExplorer":"关闭 Java 依赖项资源管理器与当前浏览文件的关联",
"contributes.commands.java.view.package.revealFileInOS": "打开所在的文件夹",
"contributes.commands.java.view.package.copyFilePath": "复制路径",
"contributes.commands.java.view.package.copyRelativeFilePath": "复制相对路径",
"configuration.java.dependency.title": "Java 依赖管理配置",
"configuration.java.dependency.showOutline": "在 Java 依赖项资源管理器中显示类成员大纲",
"configuration.java.dependency.syncWithFolderExplorer": "在 Java 依赖项资源管理器中同步关联当前打开的文件",
Expand Down
6 changes: 6 additions & 0 deletions src/commands.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,12 @@ export namespace Commands {

export const VIEW_PACKAGE_OUTLINE = "java.view.package.outline";

export const VIEW_PACKAGE_REVEAL_FILE_OS = "java.view.package.revealFileInOS";

export const VIEW_PACKAGE_COPY_FILE_PATH = "java.view.package.copyFilePath";

export const VIEW_PACKAGE_COPY_RELATIVE_FILE_PATH = "java.view.package.copyRelativeFilePath";

export const JAVA_PROJECT_CREATE = "java.project.create";

export const JAVA_PROJECT_LIST = "java.project.list";
Expand Down
6 changes: 6 additions & 0 deletions src/views/dependencyDataProvider.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,12 @@ export class DependencyDataProvider implements TreeDataProvider<ExplorerNode> {
context.subscriptions.push(commands.registerCommand(Commands.VIEW_PACKAGE_REFRESH, (debounce?: boolean) => this.refreshWithLog(debounce)));
context.subscriptions.push(commands.registerCommand(Commands.VIEW_PACKAGE_OPEN_FILE,
instrumentOperation(Commands.VIEW_PACKAGE_OPEN_FILE, (_operationId, uri) => this.openFile(uri))));
context.subscriptions.push(commands.registerCommand(Commands.VIEW_PACKAGE_REVEAL_FILE_OS,
(node: INodeData) => commands.executeCommand("revealFileInOS", Uri.parse(node.uri))));
context.subscriptions.push(commands.registerCommand(Commands.VIEW_PACKAGE_COPY_FILE_PATH,
(node: INodeData) => commands.executeCommand("copyFilePath", Uri.parse(node.uri))));
context.subscriptions.push(commands.registerCommand(Commands.VIEW_PACKAGE_COPY_RELATIVE_FILE_PATH,
(node: INodeData) => commands.executeCommand("copyRelativeFilePath", Uri.parse(node.uri))));
context.subscriptions.push(commands.registerCommand(Commands.VIEW_PACKAGE_OUTLINE,
instrumentOperation(Commands.VIEW_PACKAGE_OUTLINE, (_operationId, uri, range) => this.goToOutline(uri, range))));
Settings.registerConfigurationListener((updatedConfig, dependencyConfig) => {
Expand Down