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

"Filter" not persisted in breadcrumbs #68882

Closed
roblourens opened this issue Feb 17, 2019 · 5 comments
Closed

"Filter" not persisted in breadcrumbs #68882

roblourens opened this issue Feb 17, 2019 · 5 comments
Assignees
Labels
breadcrumbs bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@roblourens
Copy link
Member

  • Open breadcrumbs
  • Type a search query, enable "Filter on type"
  • Close breadcrumbs
  • Open again and type something
  • "Filter on type" mode is disabled, but it should remember my preference like other trees do.
@jrieken jrieken added this to the March 2019 milestone Feb 25, 2019
@jrieken jrieken added bug Issue identified by VS Code Team member as probable bug breadcrumbs labels Feb 25, 2019
@jrieken jrieken modified the milestones: March 2019, February 2019 Feb 25, 2019
@jrieken
Copy link
Member

jrieken commented Feb 25, 2019

@joaomoreno Is there some automatic trick to make the tree remember this?

@joaomoreno
Copy link
Member

@jrieken I assume you're disposing the tree when closing breadcrumbs. We can consider putting this into the tree's view state, but are you saving/restoring view state in breadcrumbs at all?

The other solution: the tree exposes the filterOnType boolean property and accepts it in the ctor options or in an updateOptions call. The tree's owner can save and restore that value.

@jrieken
Copy link
Member

jrieken commented Feb 25, 2019

I do nothing but I was hoping/asking that the use of the workspace-tree-wrapper does that...

@joaomoreno
Copy link
Member

joaomoreno commented Feb 25, 2019

@jrieken But the workspace-tree-wrapper is a subclass of tree, it also gets disposed, right? We could only do it by having ids per tree, having the ctor callers pick those ids and use them to store mementos or smth.

@jrieken
Copy link
Member

jrieken commented Feb 25, 2019

having the ctor callers pick those ids and use them to store mementos or smth.

Yeah, that was my naive assumption but I see that it lacks ids et all. I can make a breadcrumb change to persist the state

@jrieken jrieken closed this as completed Feb 26, 2019
jrieken added a commit that referenced this issue Feb 26, 2019
@octref octref added the verified Verification succeeded label Feb 26, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breadcrumbs bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants