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

enable find in the debug console #225093

Merged
merged 7 commits into from
Aug 9, 2024
Merged

enable find in the debug console #225093

merged 7 commits into from
Aug 9, 2024

Conversation

meganrogge
Copy link
Contributor

@meganrogge meganrogge commented Aug 7, 2024

fixes #4338

Follow up work will be:

find.mov

@meganrogge meganrogge marked this pull request as draft August 7, 2024 21:53
@meganrogge meganrogge self-assigned this Aug 7, 2024
@meganrogge meganrogge added this to the August 2024 milestone Aug 7, 2024
connor4312
connor4312 previously approved these changes Aug 7, 2024
Copy link
Member

@connor4312 connor4312 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@meganrogge meganrogge marked this pull request as ready for review August 8, 2024 18:00
@meganrogge meganrogge enabled auto-merge (squash) August 8, 2024 18:02
@connor4312
Copy link
Member

This gets rid of the filter entirely in the view now, right? I'm not sure that's something we'd want to just remove without giving some replacement action in its place, even if it's just an explicit button that opens the find widget

@meganrogge
Copy link
Contributor Author

fixed

Screenshot 2024-08-08 at 1 45 59 PM

@meganrogge meganrogge enabled auto-merge (squash) August 8, 2024 20:47
@meganrogge
Copy link
Contributor Author

meganrogge commented Aug 8, 2024

I just realized that currently, tree find is limited - it can only highlight matches found in the input.

However, similar to the filter widget, it can filter results based on what is found in the output.

Highlighting matches in the output might be possible when #212077 is accomplished.

Thus, by default, I think we should use filter to keep the behavior the same as before with the option to toggle to find.

@meganrogge meganrogge merged commit 8f88d20 into main Aug 9, 2024
6 checks passed
@meganrogge meganrogge deleted the merogge/de-bug branch August 9, 2024 00:21
@roblourens
Copy link
Member

I wonder about removing the filter box entirely, when the replacement doesn't support all the same features (negative patterns). Any plan to support that in the tree find widget?

@burekas7
Copy link

burekas7 commented Aug 9, 2024

Thanks.
I prefer that both features remain, the 'find' and the 'filter'

@meganrogge
Copy link
Contributor Author

meganrogge commented Aug 9, 2024

Ok, sounds good. I'll bring back the filter and will go with the default tree find keybinding, ctrlCmd+alt+f, vs ctrlCmd+f then for this feature.

@seryte
Copy link

seryte commented Sep 2, 2024

I hope it works the same as find in terminal

@meganrogge
Copy link
Contributor Author

It does not atm as mentioned on the issue. That is the ultimate goal though

@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Sep 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debug Console should support find
6 participants