Skip to content

Conversation

@h0lg
Copy link
Contributor

@h0lg h0lg commented Apr 20, 2024

Hey Fabulous Avalonia team,

in this PR I've refactored multi-binding a property into a generic and that of the AutoCompleteBox.ValueMemberBinding into a specific extension allowing to set the format and property names used in the MultiBinding and its Converter easily.

I also added an example of a remote-filtered async search to raise an issue about some erratic behaviour I noticed:
If you try completely new search terms in quick succession, you'll notice that the dropdown sometimes gets populated with results for your previous searches. You'll find some debugging in place that hopefully helps demonstrate what happens.

Thank you for your consideration!

@h0lg h0lg changed the title Auto complete async Remote-filtered async search Apr 20, 2024
@h0lg h0lg force-pushed the auto-complete-async branch from c9a08df to c416537 Compare April 22, 2024 19:50
@edgarfgp
Copy link
Member

edgarfgp commented May 7, 2024

@h0lg Thanks for improving this sample. I would be happy to merge as it is .

@h0lg h0lg force-pushed the auto-complete-async branch from c416537 to e287094 Compare May 7, 2024 22:37
@h0lg
Copy link
Contributor Author

h0lg commented May 7, 2024

@edgarfgp I have also moved the generic multiBind and the AutoCompleteBox' multiBindValue extensions into Fabulous.Avalonia for reuse - see if you want them there, otherwise feel free to revert.

@edgarfgp edgarfgp merged commit 4c659dc into fabulous-dev:main May 8, 2024
@h0lg
Copy link
Contributor Author

h0lg commented May 9, 2024

@edgarfgp Thanks for looking into this with me and improving the new extensions by adding parameter doco and simplifying!

The merged sample contains a fair amount of code left over from unsuccessful attempts to preserve tree node expansion and/or selection. I'll review and clean them up in a follow-up PR.

@h0lg h0lg deleted the auto-complete-async branch May 9, 2024 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants