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 find match in overview ruler doesn't update immediately when changing theme #145837

Closed
gregvanl opened this issue Mar 23, 2022 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal-find Relating the terminal's find widget 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 verified Verification succeeded
Milestone

Comments

@gregvanl
Copy link

gregvanl commented Mar 23, 2022

Testing #145629

On Windows 11 and Linux

Changed editor.findMatchBackground and editor.findMatchHighlightBackground:

    "workbench.colorCustomizations": {
        "editor.findMatchBackground": "#ff0000",
        "editor.findMatchHighlightBackground": "#00ff00"
    }

The color customizations worked in the regular editor but don't appear to work in the terminal find results

image

@meganrogge
Copy link
Contributor

They'll work on the next search - not for existing decorations

@meganrogge meganrogge added bug Issue identified by VS Code Team member as probable bug terminal-find Relating the terminal's find widget labels Mar 23, 2022
@meganrogge meganrogge added this to the April 2022 milestone Mar 23, 2022
@gregvanl
Copy link
Author

I've never seen the colors applied. I've tried:
start a new search
close and reopen the Find widget
delete all terminals and create a new
reload the window

@meganrogge
Copy link
Contributor

This is working for me in insider's. @gregvanl pls reopen if you can repro a case where that doesn't happen

@Tyriar
Copy link
Member

Tyriar commented Apr 1, 2022

We want to figure out how to redraw the canvas immediately when changing the theme when there are find results. The red bar is find results and should be green here:

image

@Tyriar Tyriar reopened this Apr 1, 2022
@Tyriar Tyriar changed the title Terminal Find color customizations don't appear to work Terminal find match in overview ruler doesn't update immediately when changing theme Apr 1, 2022
@meganrogge meganrogge added 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 labels Apr 4, 2022
@meganrogge
Copy link
Contributor

xtermjs/xterm.js#3729

@meganrogge
Copy link
Contributor

fixed with #147116

@rzhao271 rzhao271 added the verified Verification succeeded label Apr 28, 2022
@rzhao271
Copy link
Contributor

I tried using Greg's colours and it seems the red got applied, but not the lime:
A screenshot of the current terminal find result

I'm using the Night Owl Light theme.

@rzhao271
Copy link
Contributor

The other match results also don't seem to have any highlighting applied.

@rzhao271 rzhao271 reopened this Apr 28, 2022
@rzhao271 rzhao271 added verification-found Issue verification failed and removed verified Verification succeeded labels Apr 28, 2022
@meganrogge
Copy link
Contributor

meganrogge commented Apr 28, 2022

@rzhao271 the setting name has changed. could you pls provide a gif?

theme.mov

@meganrogge meganrogge removed the verification-found Issue verification failed label Apr 28, 2022
@rzhao271 rzhao271 added the verified Verification succeeded label Apr 28, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jun 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug terminal-find Relating the terminal's find widget 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 verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants