fluentui-blazor icon indicating copy to clipboard operation
fluentui-blazor copied to clipboard

refactor: Migrate Demo Search to use FluentAutocomplete

Open Hona opened this issue 11 months ago • 1 comments

Pull Request

📖 Description

This refactors away from Listbox -> autocomplete which is keyboard navigable.

I had to make some changes & bug fixes to the autocomplete, however this results in API changes - keen for feedback before I keep going

🎫 Issues

👩‍💻 Reviewer Notes

📑 Test Plan

✅ Checklist

General

  • [ ] I have added tests for my changes.
  • [ ] I have tested my changes.
  • [ ] I have updated the project documentation to reflect my changes.
  • [ ] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

  • [ ] I have added a new component
  • [ ] I have added Unit Tests for my new compontent
  • [ ] I have modified an existing component
  • [ ] I have validate Unit Tests for an existing component

⏭ Next Steps

Hona avatar Feb 27 '24 23:02 Hona

I've got some unit tests to write - however the rest should be good to go

Hona avatar Feb 29 '24 07:02 Hona

Sorry - been a bit busy. Going to find some time to write these tests :)

Hona avatar Mar 04 '24 22:03 Hona