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

Terminal rendering issues in Codespaces (iPadOS) #163448

Closed
alefragnani opened this issue Oct 12, 2022 · 17 comments · Fixed by #164066
Closed

Terminal rendering issues in Codespaces (iPadOS) #163448

alefragnani opened this issue Oct 12, 2022 · 17 comments · Fixed by #164066
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities ios-ipados *out-of-scope Posted issue is not in scope of VS Code terminal-input Relating to typing in the terminal not doing the right thing, IMEs not working, etc. terminal-rendering upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream
Milestone

Comments

@alefragnani
Copy link

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

  • VS Code Version: 1.73 - Codespaces / Insiders release
  • OS Version: iPadOS 15.7

Steps to Reproduce:

  1. Open a Codespaces
  2. Be sure to be using the Insiders Release
  3. Toggle Terminal
  4. The terminal rendering is unstable. Sometimes it only displays the username, sometimes is empty, and very few times it displays the entire shell (username / path / git branch)
  5. Create a new Terminal using the Terminal: Create New Terminal command
  6. A new terminal is created, but the same rendering issue occurs.
  7. You can repeat step 4 as many times as you want, and the same issue in step 5 will happen every time
  8. Type some command inside the terminal area
  9. You won’t see the keys you pressed, but hitting enter, you will notice the command itself was sent, because you can see the decorator at the right of the terminal area that refers to the new shell entry, as the picture below

49877025-E06E-4405-BEE2-E21A53EFEBC2

Switching to Stable version you won’t see such issue.

Thank you

@meganrogge
Copy link
Contributor

@rebornix could you pls see if you can repro ? I don't have an ipad

@marchellll
Copy link

It may not a problem with Codespaces, but the iPadOS. I am using code-server and got the same issue.

@meganrogge
Copy link
Contributor

Disabling terminal.integrated.gpuAcceleration should fix this. the console log from dev tools would be helpful

@alefragnani
Copy link
Author

Disabling gpuAcceleration ("terminal.integrated.gpuAcceleration": "off") indeed fix the terminal issue, thanks!

I wonder what does the Insiders release has that raises this issue, but it's good to know you found that, and probably is aware which feature is responsible for.

Thank you

@alefragnani
Copy link
Author

Not sure which piece of the console log you are looking for, but I noticed a different error message popping up when the gpuAcceleration is enabled.

image

Is that helpful?

ps: a few more details about the iPad itself. Not sure it makes any difference

  • 8th gen iPad (10.2")
  • Smart Keyboard
  • Safari browser

@meganrogge meganrogge added this to the Backlog milestone Oct 19, 2022
@Tyriar
Copy link
Member

Tyriar commented Oct 19, 2022

Very helpful, not sure right now how that could happen though. Code it's pointing at: https://github.com/xtermjs/xterm.js/blob/1b13127f0be68b7e4b1a6c71717c74f503ff783d/addons/xterm-addon-canvas/src/BaseRenderLayer.ts#L374-L384

@Tyriar
Copy link
Member

Tyriar commented Oct 19, 2022

Sigh... https://bugs.webkit.org/show_bug.cgi?id=149990

@Tyriar
Copy link
Member

Tyriar commented Oct 19, 2022

Tracking upstream: xtermjs/xterm.js#4218

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream and removed confirmation-pending labels Oct 19, 2022
@Tyriar Tyriar modified the milestones: Backlog, October 2022 Oct 19, 2022
@Tyriar Tyriar added the upstream-issue-fixed The underlying upstream issue has been fixed label Oct 19, 2022
@alefragnani
Copy link
Author

Any idea why this is happening only in the Insiders? Because the
gpuAcceleration doesn't seems to be exclusive to Insiders.

@Tyriar
Copy link
Member

Tyriar commented Oct 19, 2022

@alefragnani the canvas renderer just got a major refactor to share a lot of code with the webgl renderer.

Tyriar added a commit that referenced this issue Oct 19, 2022
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Oct 19, 2022
@vscodenpa vscodenpa added the insiders-released Patch has been released in VS Code Insiders label Oct 20, 2022
@roblourens roblourens added the author-verification-requested Issues potentially verifiable by issue author label Oct 27, 2022
@vscodenpa
Copy link

This bug has been fixed in the latest release of VS Code Insiders!

@alefragnani, you can help us out by commenting /verified if things are now working as expected.

If things still don't seem right, please ensure you're on version 9ad0693 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.

Happy Coding!

@meganrogge
Copy link
Contributor

@rebornix can you try to verify this on your ipad?

@alefragnani
Copy link
Author

Hi, I just tested and the issue remains.

28177EB7-E9F0-4186-B112-4F48357F9276

I’ll try again tomorrow, and post the updates here.

Thank you

@meganrogge meganrogge reopened this Oct 28, 2022
@vscodenpa vscodenpa removed the insiders-released Patch has been released in VS Code Insiders label Oct 28, 2022
@Tyriar
Copy link
Member

Tyriar commented Nov 23, 2022

We may need a check like this somewhere for canvas too?

https://github.com/xtermjs/xterm.js/blob/452241ca9207506b8fb850b3062e5b4dbcf568a4/addons/xterm-addon-webgl/src/WebglAddon.ts#L36-L38

I don't totally understand what's going on here and don't have an iPad.

@Tyriar Tyriar modified the milestones: November 2022, Backlog Nov 23, 2022
lemanschik pushed a commit to code-oss-dev/code that referenced this issue Nov 25, 2022
This was referenced Nov 30, 2022
@Tyriar Tyriar added help wanted Issues identified as good community contribution opportunities and removed author-verification-requested Issues potentially verifiable by issue author upstream-issue-fixed The underlying upstream issue has been fixed labels Dec 2, 2022
@Tyriar Tyriar added the terminal-input Relating to typing in the terminal not doing the right thing, IMEs not working, etc. label Dec 9, 2022
@jet082
Copy link

jet082 commented May 26, 2023

This is marked as a duplicate of #144232, but is that really the case? This seems to be about graphical rendering whereas #144232 is about input problems. I would leave a comment there, but it is closed to contributions. I propose that is reopened and this issue is kept to the rendering issues.

@Tyriar
Copy link
Member

Tyriar commented May 26, 2023

@jet082 I reopened and unlocked it

@Tyriar
Copy link
Member

Tyriar commented Dec 10, 2024

The canvas renderer has been removed from xterm.js/vscode, so this is no longer relevant 🎉

@Tyriar Tyriar closed this as completed Dec 10, 2024
@Tyriar Tyriar added the *out-of-scope Posted issue is not in scope of VS Code label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities ios-ipados *out-of-scope Posted issue is not in scope of VS Code terminal-input Relating to typing in the terminal not doing the right thing, IMEs not working, etc. terminal-rendering upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants