question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Revisit the use of `ITextEditorService` in preferences component

See original GitHub issue

We want to move away from allowing anyone to create heavy typed editor inputs. One clear method we need to deprecate is IEditorService.createEditorInput. The usages we have today need to be changed to untyped inputs.

We seem to be having a few places left:

Preferences @roblourens

image

Custom Editors

image

Text Editors

image

We then have quite a few more usages that imho are less critical because they originate from contexts where we actually do want typed inputs (editor input deserialisation). However, we should probably move those into respective components and out of the editor service itself. I will create a follow up issue on that.

Issue Analytics

  • State:open
  • Created 2 years ago
  • Comments:21 (21 by maintainers)

github_iconTop GitHub Comments

1reaction
bpaserocommented, Dec 17, 2021

Settings editor is the last remaining one.

1reaction
roblourenscommented, Jun 23, 2021

Will be deleted first thing in debt week when I get rid of the split json editor 😁

Read more comments on GitHub >

github_iconTop Results From Across the Web

No results found

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found