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

Test proposed custom editor API #96315

Closed
1 of 3 tasks
mjbvz opened this issue Apr 27, 2020 · 7 comments
Closed
1 of 3 tasks

Test proposed custom editor API #96315

mjbvz opened this issue Apr 27, 2020 · 7 comments
Assignees
Labels
insiders-released Patch has been released in VS Code Insiders testplan-item
Milestone

Comments

@mjbvz
Copy link
Collaborator

mjbvz commented Apr 27, 2020

Refs #77131

Complexity: 5

Create Issue


Test using the updated custom editor API proposal from #77131. To be clear, this is the proposal for full custom editors (such as for binary editors) and not the already finalized api for custom text editors

Overview
This iteration we again reworked the custom editor API proposal to better align it with the notebook api and also help implementers. For this exploratory test plan item, your goal is to try using the API and try to find bugs in our custom editor implementation. I've intentionally left this open ended.

Areas to focus on:

  • Do custom editors behave like normal editors for save, edit, and revert?

  • Try using supportsMultipleEditorsPerDocument with your custom editor.

    • When this setting is disabled, ensure you can't get into a state where more than one copy of the editor is opened for a single document.
    • When it is disabled, make sure splitting and other operations work as expected
  • Backup

    • Make sure your extension can back up the resource and that these are properly restored
    • Make sure VS Code calls dispose on the backup when you would expect

Resources
Sample custom editor extension to build on (it's the "paw draw" editor): https://github.com/mjbvz/vscode-extension-samples/tree/custom-editor-examples

@roblourens
Copy link
Member

@mjbvz which backups are referred to here? It looks like vscode decides when CustomDocumentBackups are deleted, is that correct?

 * During `revert`, your extension should also clear any backups for the custom editor. Backups are only needed
* when there is a difference between an editor's state in VS Code and its save state on disk.

@mjbvz
Copy link
Collaborator Author

mjbvz commented Apr 28, 2020

That's actually an out of date comment. VS Code will now call .delete on the current backup for you. I'll fix the comment

@alexdima
Copy link
Member

Here is my status on this: I started testing and then looked to integrate a hex editor into vs code, but I couldn't find one that works well, so I started building one. But building one is taking quite some time. So, this item will leak through to the debt week for me, I'm sorry about that.

If you want more timely feedback, please also assign someone else.

@roblourens
Copy link
Member

I also tried to build a real custom editor, then realized that I would spend more time with my code than the API 😆

@deepak1556
Copy link
Collaborator

In my case, I am building a custom editor for chromium cache files <User-data-dir>/Cache and I can't seem to finish it in the testing week. I will also continue exploring in debt week. For now I can sign-off with verifying the sample extension provided, if thats required.

@connor4312 connor4312 modified the milestones: April 2020, May 2020 May 7, 2020
@roblourens roblourens removed their assignment Jun 2, 2020
@roblourens
Copy link
Member

This one shouldn't have had its milestone moved. Did you finish testing @alexdima and @deepak1556? We have 5 extra points now.

@mjbvz mjbvz modified the milestones: May 2020, April 2020 Jun 2, 2020
@mjbvz
Copy link
Collaborator Author

mjbvz commented Jun 2, 2020

I think this one should be left in the April milestone. I created a new test plan item for this iteration: #99003

@mjbvz mjbvz closed this as completed Jun 2, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
insiders-released Patch has been released in VS Code Insiders testplan-item
Projects
None yet
Development

No branches or pull requests

8 participants
@roblourens @eamodio @deepak1556 @connor4312 @alexdima @sandy081 @mjbvz and others