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

Unify colors of find in file and workspace search results #9955

Closed
chrmarti opened this issue Jul 29, 2016 · 3 comments
Closed

Unify colors of find in file and workspace search results #9955

chrmarti opened this issue Jul 29, 2016 · 3 comments
Assignees
Labels
*as-designed Described behavior is as designed editor-find Editor find operations polish Cleanup and polish issue search Search widget and operation issues ux User experience issues verified Verification succeeded
Milestone

Comments

@chrmarti
Copy link
Contributor

  • VSCode Version: Code - Insiders 1.4.0-insider (4656fe3, 2016-07-29T05:03:52.605Z)
  • OS Version: Darwin x64 15.5.0

Find in file (Ctrl+F) and workspace search (Ctrl+Shift+F) results are colored not exactly the same. It might be easier to read the color-coding if it was the same.

Local search:
localsearch

Global search:
globalsearch

Note the different browns for the results and the different renderings of the current line in the two cases.

(Extracted from #6402.)

@kieferrm kieferrm added ux User experience issues themes Color theme issues search Search widget and operation issues labels Aug 2, 2016
@alexdima alexdima added this to the Backlog milestone Aug 2, 2016
@alexdima alexdima added polish Cleanup and polish issue bug Issue identified by VS Code Team member as probable bug editor-find Editor find operations labels Aug 18, 2016
@alexdima alexdima removed their assignment Aug 29, 2016
@alexdima alexdima modified the milestones: August 2016, Backlog Aug 29, 2016
@sandy081
Copy link
Member

This happens when both Search and Find are active on file for the same term. Otherwise they are similar in colors for results. If both are active then each adds same decoration and there are two overlay decoration on the same word which is causing a different color. One soultion for this could be to disallow duplicate decorations but I think this is complex as state is going to be involved. Also it has to be dealt properly when a decoration is removed.

IMO the current behavior is reasonable.

aug-31-2016 17-14-06

aug-31-2016 17-14-21

It leaves to the issue about Search selecting the line of current find result. I will create a separate issue for that and leave this issue only for search and find results color.

@sandy081 sandy081 added *as-designed Described behavior is as designed and removed themes Color theme issues labels Aug 31, 2016
@octref
Copy link
Contributor

octref commented Sep 1, 2016

Since this is as-designed, there isn't anything to verify.
I'm adding the verified tag and removing bug.
@sandy081 Let me know if I'm doing it wrong.

@octref octref added verified Verification succeeded and removed bug Issue identified by VS Code Team member as probable bug labels Sep 1, 2016
@Tyriar
Copy link
Member

Tyriar commented Sep 1, 2016

Related discussion: #11354

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*as-designed Described behavior is as designed editor-find Editor find operations polish Cleanup and polish issue search Search widget and operation issues ux User experience issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants