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

Cannot show "Compare Selected" menu item for two unsaved "Untitled" files in View -> Explorer #56316

Closed
johnnytemp opened this issue Aug 14, 2018 · 10 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@johnnytemp
Copy link

  • VSCode Version: 1.26.0
  • OS Version: macOS 10.13.6

Steps to Reproduce:

  1. Cmd-N twice to New two files, type "abc" in each document.
  2. menu View -> Explorer, focus in "Open Editors" and select only the two new files, right click over "one of them", there should be "Compare Selected" menu item. However, it is missing. Only these 4 items is showing now: "Close", "Close Others", "Close Saved", "Close All".

(An in-pretty workaround I found is to right-click over another saved file when the two "Untitled" files selected. However, it is difficult for users to find this trick, and should need a real fix)

Does this issue occur when all extensions are disabled?: Yes

@johnnytemp
Copy link
Author

Seems reported and maybe fixed in 1.27.0 preview. Will wait to try if ok in next version.

@johnnytemp
Copy link
Author

But it is strange that until 1.26.0 this issue is still un-fixed, as the issue has been reported in like #42598 ? Will it really work on next version?

@johnnytemp
Copy link
Author

Because previous tickets about similar issue are closed around "Jul 2017", but 2018 July version (1.26.0) still has this bug. I suggest to keep this issue open to avoid other users creating same ticket, until a released version is launched, and the bug really fixed.

Thank you.

@johnnytemp johnnytemp reopened this Aug 14, 2018
@isidorn
Copy link
Contributor

isidorn commented Aug 15, 2018

This works for me in latest stable and latest vscode insiders on os x
Please let me know what you are doing differently than me

compare

@isidorn isidorn added the info-needed Issue requires more information from poster label Aug 15, 2018
@johnnytemp
Copy link
Author

johnnytemp commented Aug 17, 2018

Although I tested it work in latest vscode insiders preview, just worry that such code will be release in next minor version or not. (Since the related tickets are closed long ago, but bug still not fixed in the recent released version). Welcome to close it, and if next minor version not work, I will reopen it.

Thank you.

@vscodebot vscodebot bot removed the new release label Aug 21, 2018
@wthenrichs
Copy link

I am also experiencing this issue.
I'm exclusively using the Insiders preview.
The issue only appears if I open a new empty window of vs code. If I open a folder, I'm able to compare both saved, and unsaved files.
If it helps the context menu for the windows which have this problem only show 5 options:
Save As...
Close
Close Others
Close Saved [Ctrl+K U]
Close All [Ctrl+K W]

Current Versions:
Version: 1.27.0-insider
Commit: 346018a
Date: 2018-08-24T05:19:27.628Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64

Please let me know if I can provide any more information.

@isidorn
Copy link
Contributor

isidorn commented Aug 24, 2018

@wthenrichs can you please provide a gif, or a picture of what you are exactly doing. Step by step would also help

@wthenrichs
Copy link

wthenrichs commented Aug 24, 2018

@isidorn
I've attached two gifs.
The first is of a new vscode window that is empty.
The second is a vscode windows which was empty but I opened a folder via File > Open Folder.
The folder is called demo and is an empty folder.

new_window_no_compare

open_folder_compare

@isidorn
Copy link
Contributor

isidorn commented Aug 24, 2018

@wthenrichs thanks for the lovely gifs.

@bpasero @jrieken this issue seems to be that canHandleResource('untitled') always returns false. How should we handle the untitled case?
@jrieken this seems to have broken this 76f2588

@isidorn isidorn added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Aug 24, 2018
@isidorn isidorn added this to the August 2018 milestone Aug 24, 2018
@jrieken
Copy link
Member

jrieken commented Aug 24, 2018

@jrieken this seems to have broken this 76f2588

No - the untitled schema has never been handled by the file system service, there is no provider for it but there exists something like an untitled service

@jrieken jrieken removed their assignment Aug 24, 2018
@bpasero bpasero added the verified Verification succeeded label Aug 30, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants