Selection Prompt Search

[slang25]

This PR adds a couple of features to Selection Prompt:

• Search as you type
• Skipping selection of group nodes for Leaf selection mode

I've yet to add the tests which is a bit naughty, but wanted to share what I have so far to start some discussion. Is this feature welcome, or is there a better direction?

[slang25]

This is somewhat related to #1258, as this is a feature I'd really like in my CLI, however my only feasible options are to fork the library or hope I can get this merged.

[GOATS2K]

This looks incredibly helpful - I'd love to see this merged soon.

[slang25]

Thanks @GOATS2K, I'm pretty happy with it and am using it in one of my apps. I've just updated the API names to be a bit more sensible.

@patriksvensson what are your thoughts on this?

[patriksvensson]

This looks good, but I think some minor things need to be addressed. I would also like to see some tests for the search functionality.

[slang25]

@patriksvensson thanks for taking a look, I've addressed the feedback. I've added some tests, I could add some Verify tests too and cover more scenarios but am on the fence, let me know what you think.

[sleepyfran]

Thanks a lot for this, really excited to use it! I took it for a spin in a personal fork and found the issue I commented about. Other than that it seems to work perfectly 🙇🏻

[slang25]

Hi @patriksvensson, please could you take another look at this PR when you get some time. I'm happy to address any further feedback 🙂

[Jacob-32587]

Would love to see this go through soon, extremely useful feature when dealing with large prompts.

[patriksvensson]

@slang25 I'm rebasing this and fixing the merge conflicts right now

[patriksvensson]

Merged! Thank you for your contribution. Much appreciated! 👍

[slang25]

Thanks @patriksvensson, this library is such a gem for the .NET community, I hope you have time to keep it going 🙂 I was worried we'd lose you to swift 😅

[patriksvensson]

@slang25 Haha, thanks! I still have a lot of love for dotnet 🙂

[fihorvat]

@slang25 Sorry for the late comment but it looks like there is a bug. If you try to use something other than string for SelectionPrompt<T> in combination with the UseConverter the cursor selection doesn't work how it should.
Text highlighting works but the cursor doesn't move.

Here is a simple test data:

class TestItem
{
  public string Name { get; set; }
  public string Url { get; set; }
}

var items = new TestItem[] {
  new() { Name = "banana", Url = "something" },
  new() { Name = "orange", Url = "something" },
  new() { Name = "apple", Url = "something" }
};

var prompt = new SelectionPrompt<TestItem>()
  .Title("Select [green]fruit[/]?")
  .PageSize(10)
  .AddChoices(items)
  .UseConverter(x => x.Name)
  .EnableSearch();
prompt.SearchHighlightStyle(Style.Parse("yellow1"));
var response = AnsiConsole.Prompt(prompt);

And could you please add the method SearchHighlightStyle same as we have for HighlightStyle?

[patriksvensson]

@fihorvat This PR had been merged and no further work is expected by the contributor. Please open a new issue if you experience any bugs.