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

Search 'collapse' button no longer works #204316

Closed
Dustin-Digitar opened this issue Feb 5, 2024 · 9 comments · Fixed by #205235
Closed

Search 'collapse' button no longer works #204316

Dustin-Digitar opened this issue Feb 5, 2024 · 9 comments · Fixed by #205235
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders search Search widget and operation issues verified Verification succeeded
Milestone

Comments

@Dustin-Digitar
Copy link

Type: Bug

  1. Open the search side panel
  2. Enter a search term
  3. The search returns results
  4. Try and click the 'collapse all' button at the top right of the search panel
  5. Observe: nothing happens. Also, the icon shows a '+'. It should show a '-' because the list is already expanded.

Before the latest update, this worked as expected.

VS Code version: Code 1.86.0 (0504748, 2024-01-31T10:28:19.990Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info
Item Value
CPUs AMD Ryzen 7 5825U with Radeon Graphics (16 x 1996)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
Load (avg) undefined
Memory (System) 38.83GB (19.29GB free)
Process Argv
Screen Reader no
VM 0%
Extensions (32)
Extension Author (truncated) Version
Bookmarks ale 13.4.2
path-intellisense chr 2.8.5
vscode-markdownlint Dav 0.54.0
vscode-eslint dba 2.4.4
githistory don 0.6.20
prettier-vscode esb 10.1.0
auto-close-tag for 0.5.15
copilot Git 1.156.0
copilot-chat Git 0.12.0
remotehub Git 0.62.0
vscode-pull-request-github Git 0.80.0
vscode-graphql-syntax Gra 1.3.1
path-autocomplete ion 1.25.0
vscode-env Iro 0.1.0
rainbow-csv mec 3.11.0
debugpy ms- 2024.0.0
remote-wsl ms- 0.86.0
remote-repositories ms- 0.40.0
vsliveshare ms- 1.0.5905
peggy-language Peg 2.4.1
language-plan und 0.0.1
excalidraw-editor pom 3.7.1
quicktype qui 12.0.46
vscode-yaml red 1.14.0
vscode-paste-and-indent Rub 0.0.8
partial-diff ryu 1.4.3
multi-command ryu 1.6.0
intellicode-api-usage-examples Vis 0.2.8
vscodeintellicode Vis 1.2.30
vscode-icons vsc 12.7.0
gitblame wad 10.7.1
copy-hover-type You 0.0.7
@Dustin-Digitar
Copy link
Author

image

@Dustin-Digitar
Copy link
Author

This is a pretty significant bug, because it makes the search near useless when there are a large number of files containing hits, or when there are many hits in a single file. It makes the list very tall to scroll through, and file names are harder to pick out.

@andreamah
Copy link
Contributor

Hmm, I can't seem to repro this issue. What value do you have for search.collapseResults? Do you have any other search-related settings?

@andreamah andreamah added the info-needed Issue requires more information from poster label Feb 5, 2024
@ejohnso49
Copy link

I am able to reproduce:

Version: 1.86.0 (Universal)
Commit: 0504748
Date: 2024-01-31T10:29:15.765Z
Electron: 27.2.3
ElectronBuildId: 26495564
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: 11.8.172.18-electron.0
OS: Darwin arm64 23.2.0

Looking at the option that was mentioned:
search.collapseResults: "alwaysExpand"

When I search, after results are populated I am not able to collapse them using the button shown in the screenshot.

Workaround
I seem to have restored functionality by modifying the search.collapse to a different value than alwaysExpand. I unfortunately don't know which specific value fix things, so would recommend trying them each, redo the search, and then attempt to click the collapse button. I am now able to toggle the button regardless of what option search.collapseResults is set to

@andreamah
Copy link
Contributor

Hmm. Does your settings.json explicitly say search.collapseResults: "alwaysExpand" now? and if you remove it, can you reproduce the bug?

@ejohnso49
Copy link

It does not have that explicitly defined in my case and I don't believe I had it explicitly set previously. I also have since updated to 1.86.1. I'm having a hard time reliably reproducing the bug unfortunately. I will make sure to report back if I reproduce. I do wonder if some of the behavior was less due to the button and maybe the search still loading?

@gjsjohnmurray
Copy link
Contributor

I have submitted a PR that I think fixes the issue.

@andreamah andreamah added bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues and removed info-needed Issue requires more information from poster labels Feb 14, 2024
@andreamah andreamah added this to the February 2024 milestone Feb 14, 2024
@VSCodeTriageBot VSCodeTriageBot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Feb 14, 2024
@eleanorjboyd eleanorjboyd added the verified Verification succeeded label Feb 21, 2024
@Anutrix
Copy link

Anutrix commented Feb 28, 2024

Waiting for this since a few days. I assume the fix is isn't in 1.86.2.

@gjsjohnmurray
Copy link
Contributor

@Anutrix no, it's in 1.87 which should be published this week.

@microsoft microsoft locked and limited conversation to collaborators Jun 10, 2024
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 insiders-released Patch has been released in VS Code Insiders search Search widget and operation issues verified Verification succeeded
Projects
None yet
7 participants