#98942 Clear Search Pattern Fields#100024
#98942 Clear Search Pattern Fields#100024roblourens merged 4 commits intomicrosoft:masterfrom edjeffreys:issue/98942-search-clear-exclude
Conversation
Finally realized what a 2-stage button is... Clears search fields first if there are any values entered, otherwise clears file pattern fields
|
Thanks! |
|
I wouldn't create a separate icon for this, we already do this in the Settings & Keyboard Shortcut search input (though it's a single input). I'd ensure you update the title properly. I also think this should be behind a setting as people will be split on a preference. I'd personally want it to clear everything on a single click. |
|
Happy to take a look at both the title, and the setting. Any preference on what the default behaviour should be for clearing? |
roblourens
left a comment
There was a problem hiding this comment.
Looks great! I don't want to add a setting yet, since the default behavior is not changing, and I try to avoid adding settings for every little thing. We can wait and see if we get more feedback about that though.
|
@JacksonKearl what are you thinking that the title should be updated to say? |
|
Thanks @roblourens! I'll hold fire for now then. |
|
I don't know that we need to update the title, I think that it already will say "Clear Search Results" when there are no search results and it is just clearing the input box. If we want to be totally specific, I think there would 3 states.
But I don't know that it's necessary, what do you think @JacksonKearl? Also I think we're fine without extra icons. |
|
I'm convinced; people will probably discover it anyways if they need it. |
|
Just to be clear, this is the behaviour I see in the Insiders' Build: The first click clears the search results and the query/replace terms and does not affect the The second click clears the I like this, it just wasn't clear from the preceding comments that this was how it would turn out. |
Edit (Took me longer than it should have to realise what a 2-stage button means...)
This PR fixes #98942