It should also follow the browser's default behavior:Ctrl+Click /Command+Click means "Open in New Window"

Done, thanks for pointing this out

By doing some searches, I think "middle key" is not a widely used approach to "open a new window". I do not see browsers officially support it.

The widely supported approach is "Ctrl+LeftClick"

If you'd like to support "middle click", the ideal approach can be like this:

• make the items to be real<a> links, then middle-click works for users by default.
• use our@click handler to handlectrl/meta keys, and stop the default link's behavior.

I think we should just render a native<a> element and onlypreventDefault theclick event withbutton 0, while letting the browser handle all other interactions.

This way, stuff like middle click, ctrl+click, cmd+click will all work natively and don't even need explicit handlers.

I think we should just render a native<a> element and onlypreventDefault theclick event withbutton 0, while letting the browser handle all other interactions.

This way, stuff like middle click, ctrl+click, cmd+click will all work natively and don't even need explicit handlers.

The same as#34730 (comment) ? Or is there something I missed?

I think we should just render a native<a> element and onlypreventDefault theclick event withbutton 0, while letting the browser handle all other interactions.
This way, stuff like middle click, ctrl+click, cmd+click will all work natively and don't even need explicit handlers.

The same as#34730 (comment) ? Or is there something I missed?

Basically the same, but I guess the code still need to check whether any modifier key is held, and onlye.preventDefault when none are because deciding only one.button is not enough.

I see, maybe ideally it could be like this:

• use<a> for tree items with real links.
• if no ctrl/meta/alt key, then use our "fetch" to load the file view content without reload, and preventDefault.
• otherwise, we do nothing and leave the work to the browser.

Yes, so basically:

if(e.button!==0||e.ctrlKey||e.metaKey||e.altKey||e.shiftKey){return;// let browser handle the click}e.preventDefault();// update view to clicked file

MaybeshiftKey can be removed, I'm not aware of any special functionality tied to it.altKey is useful for text selection within a link.

Thanks for the suggestions, I've updated the PR accordingly by using<a> tags.

By doing some searches, I think "middle key" is not a widely used approach to "open a new window". I do not see browsers officially support it.

Interesting, I almost exclusively use the middle click to open links and such in a new tab.

Maybe shiftKey can be removed, I'm not aware of any special functionality tied to it. altKey is useful for text selection within a link.

Shift+LeftClick opens the link in a new browser window, so I've kept it.

Yes, we need to keep all modifiers, they all have a purpose and mine opens new window on shift+click too. middle click and ctrl+click are both known to me, so they are not so obscure :)

It seems when a subtree is loaded, the whole screen flashes briefly.

Fixed