Open With... not working on folders

[bhufmann]

Bug Description:

Open With -> Trace Viewer of the Theia Trace Viewer doesn't work anymore starting with Theia version 1.50.0. This feature allowed users to open traces that are files as well as traces that are folders. The limitation should be put back or alternatively make OpenWidthHandler configurable for files or folders or both (default files only):

this.openWithService.registerHandler({
            id: this.id + '-open-with',
            label: this.label,
            providerName: 'Theia Trace Viewer',
            appliesToFiles: true, // <----- suggestion
            appliesToFolders: true, // <----- suggestion
            canHandle: (uri: URI) => this.canHandle(uri),
            open: (uri: URI) => this.open(uri)
        });

Steps to Reproduce:

1. Build and run the Theia trace viewer with PR https://github.com/eclipse-cdt-cloud/theia-trace-extension/pull/1087
2. Right mouse-click on a folder
3. "Open With..." is not shown

Additional Information

• Operating System: All
• Theia Version: 1.50.0
• related change: #13678

[JonasHelming]

@msujew

[msujew]

Sure, that sounds pretty reasonable to me. @bhufmann Interested in contributing something for this?

[bhufmann]

I'm not really familiar with Theia internals. It would be probably faster if you @msujew implement it :-)

I had a quick look, and thought it would be just updating getHandler() and check if the URI points to folder and check if the handler is configured for folders and then include it or not. What is a quick way to check if the uri points to a file or folder? Please let me know.