-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Make the DOM renderer the default and move the canvas renderer into an addon #3271
Comments
Commenting here as microsoft/vscode#120131 & microsoft/vscode#106202 are locked. On latest build, I noticed a lot of lag on the integrated terminal, previously I was using EDIT: Actually, even setting |
👍 for that idea. Imho it is ok to have just one default renderer in the base bundle, and the logical choice would be the DOM renderer. It has a much smaller code footprint and works well enough for casual integrations. Few notes though:
|
Hoping to get time for this in the coming weeks. |
Actually this would have to go in a v5, maybe it's time 🤔 |
No need to load the code if it's not being used. VS Code has recently transitioned to webgl as the default: microsoft/vscode#106202
Note that I decided we should probably keep the canvas renderer since otherwise Safari/iPad/etc. are stuck with the much slower DOM renderer. The canvas renderer is much better than that and doesn't really need much maintenance at this point.
The text was updated successfully, but these errors were encountered: