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

Persistent background peeking through certain text #3970

Closed
npezza93 opened this issue Jul 30, 2022 · 12 comments · Fixed by #3973 or #3989
Closed

Persistent background peeking through certain text #3970

npezza93 opened this issue Jul 30, 2022 · 12 comments · Fixed by #3973 or #3989
Assignees
Labels
area/addon/webgl type/bug Something is misbehaving
Milestone

Comments

@npezza93
Copy link
Contributor

Seems as though on v5 some text has a different background that isn't present in v4 and persists through selection.

Details

  • Browser and browser version: electron 19
  • OS version: macos monterey
  • xterm.js version: v5 beta

Steps to reproduce

I've so far only noticed this when yarning so im not sure if v4 was just displaying text incorrectly all along and a bug got fixed.
v4:

Screen.Recording.2022-07-30.at.10.31.28.AM.mov

v5:

Screen.Recording.2022-07-30.at.10.32.03.AM.mov
@Tyriar
Copy link
Member

Tyriar commented Jul 30, 2022

@npezza93 do you know what's causing this background color?

image

Turning on logLevel = 'debug' will let you see the sequence.

Also does this happen on all renderers?

@Tyriar Tyriar added this to the 5.0.0 milestone Jul 30, 2022
@npezza93
Copy link
Contributor Author

Also does this happen on all renderers?

Looks like just webgl. Canvas is fine

@npezza93
Copy link
Contributor Author

npezza93 commented Jul 30, 2022

Heres the debug log:
image

@Tyriar
Copy link
Member

Tyriar commented Jul 30, 2022

Hmm, I think devtools parsing and hiding the styling sequence I care about. Can you enable the dom renderer and inspect one of the problem cells instead?

@npezza93
Copy link
Contributor Author

npezza93 commented Jul 30, 2022

@Tyriar I'm gonna guess something to do with: #3932

image

@Tyriar
Copy link
Member

Tyriar commented Jul 30, 2022

Thanks, I think I know what's going on

@Tyriar Tyriar added type/bug Something is misbehaving area/addon/webgl and removed needs more info labels Jul 30, 2022
Tyriar added a commit to Tyriar/xterm.js that referenced this issue Jul 30, 2022
@npezza93
Copy link
Contributor Author

npezza93 commented Jul 31, 2022

Still seeing the different background on the lastest master:
Canvas(correct):
image
Webgl:
image

It does go away on selection now though.

Note the [1/4]and $ ./node_modules/.bin/electron .

@Tyriar
Copy link
Member

Tyriar commented Jul 31, 2022

@npezza93 is this with allow transparency enabled?

@npezza93
Copy link
Contributor Author

@Tyriar Nope, allowTransparency is false

@Tyriar
Copy link
Member

Tyriar commented Jul 31, 2022

@npezza93 hm, what theme are you using? I don't think I saw this when I fixed the selection problem yesterday

@npezza93
Copy link
Contributor Author

image

Copy pastable json string version:

'{"fontFamily":"JetBrainsMono Nerd Font Mono","fontSize":15,"fontWeight":"normal","cursorStyle":"block","cursorBlink":true,"scrollback":4000,"macOptionIsMeta":true,"hideContextMenu":true,"rightClickSelectsWord":true,"macOptionClickForcesSelection":true,"theme":{"green":"rgb(171, 233, 179)","brightGreen":"rgb(171, 233, 179)","black":"rgb(110, 108, 126)","brightBlack":"rgb(152, 139, 162)","red":"rgb(242, 143, 173)","brightRed":"rgb(242, 143, 173)","yellow":"rgb(250, 227, 176)","brightYellow":"rgb(250, 227, 176)","blue":"rgb(150, 205, 251)","brightBlue":"rgb(150, 205, 251)","magenta":"rgb(245, 194, 231)","brightMagenta":"rgb(245, 194, 231)","cyan":"rgb(137, 220, 235)","brightCyan":"rgb(137, 220, 235)","white":"rgb(217, 224, 238)","brightWhite":"rgb(217, 224, 238)","cursor":"rgb(248, 189, 150)","cursorAccent":"rgb(30, 29, 47)","foreground":"rgb(217, 224, 238)","background":"rgb(30, 29, 47)","selectionBackground":"rgba(217, 224, 238, 0.2)"},"allowTransparency":false,"cursorWidth":2,"fontWeightBold":"bold","lineHeight":1,"letterSpacing":"0","tabStopWidth":8,"allowProposedApi":true,"logLevel":"debug"}'

@Tyriar
Copy link
Member

Tyriar commented Jul 31, 2022

Can repro 👍

term.options.theme = {"green":"rgb(171, 233, 179)","brightGreen":"rgb(171, 233, 179)","black":"rgb(110, 108, 126)","brightBlack":"rgb(152, 139, 162)","red":"rgb(242, 143, 173)","brightRed":"rgb(242, 143, 173)","yellow":"rgb(250, 227, 176)","brightYellow":"rgb(250, 227, 176)","blue":"rgb(150, 205, 251)","brightBlue":"rgb(150, 205, 251)","magenta":"rgb(245, 194, 231)","brightMagenta":"rgb(245, 194, 231)","cyan":"rgb(137, 220, 235)","brightCyan":"rgb(137, 220, 235)","white":"rgb(217, 224, 238)","brightWhite":"rgb(217, 224, 238)","cursor":"rgb(248, 189, 150)","cursorAccent":"rgb(30, 29, 47)","foreground":"rgb(217, 224, 238)","background":"rgb(30, 29, 47)","selectionBackground":"rgba(217, 224, 238, 0.2)"}

Then in terminal:

echo -e '\x1b[2mdim text\x1b[0m'

@Tyriar Tyriar reopened this Jul 31, 2022
@Tyriar Tyriar self-assigned this Jul 31, 2022
Tyriar added a commit to Tyriar/xterm.js that referenced this issue Jul 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/addon/webgl type/bug Something is misbehaving
Projects
None yet
2 participants