Skip to content

Include custom editors in history#83019

Merged
mjbvz merged 1 commit intomicrosoft:masterfrom
mjbvz:include-custom-editors-in-history
Oct 22, 2019
Merged

Include custom editors in history#83019
mjbvz merged 1 commit intomicrosoft:masterfrom
mjbvz:include-custom-editors-in-history

Conversation

@mjbvz
Copy link
Collaborator

@mjbvz mjbvz commented Oct 21, 2019

Fixes #81824

The history service currently drops any editor inputs that are not instances FileEditorInput. This check does not work for custom editors, which are backed by resources like a FileEditorInput but do not subclass FileEditorInput

This fix instead creates history entries for any input that has a resource that is backed by the file-service

Fixes microsoft#81824

The history service currently drops any editor input that are not `FileEditorInput`. This does not work for custom editors, which are always backed by a resource but do not subclass `FileEditorInput`

The fix here is to create history entries for any input that has a resource that is backed by the file-service
@mjbvz mjbvz requested a review from bpasero October 21, 2019 22:55
@mjbvz mjbvz self-assigned this Oct 21, 2019
@bpasero
Copy link
Member

bpasero commented Oct 22, 2019

Yeah, that is probably the right check to do.

@mjbvz mjbvz added this to the October 2019 milestone Oct 22, 2019
@mjbvz mjbvz merged commit 9d752a2 into microsoft:master Oct 22, 2019
@mjbvz mjbvz deleted the include-custom-editors-in-history branch October 22, 2019 18:07
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Webview editor doesn't appear in Open Previous Editor From History command

2 participants