fluentui icon indicating copy to clipboard operation
fluentui copied to clipboard

SearchBox

Open paulgildea opened this issue 2 years ago • 9 comments

💡 When you create a PR for any of the checklist items, add a link to this Epic under the PR's Related Issues section.

Preparation

Implementation

Validation

paulgildea avatar Feb 03 '23 18:02 paulgildea

Karina to provide design support.

ADO Task

briandrouin avatar Oct 09 '23 20:10 briandrouin

📢 "In preview searchbox"

gouttierre avatar Oct 30 '23 11:10 gouttierre

@gouttierre per Karina initial Searchbox and dropdown completed from design, but the dropdown search result component still needs to be built. We can discuss more in our 1:1 on Wednesday.

briandrouin avatar Oct 31 '23 20:10 briandrouin

📢 "Component is in preview. Karina has finished designing the initial Searchbox and dropdown, but the dropdown search result component is yet to be built."
"The design specifications are ready, and @JustSlone will follow up with @smhigley to evaluate how easy it will be to implement the required functionality. "

gouttierre avatar Nov 20 '23 12:11 gouttierre

📢 "Waiting on engineering to respond about the next steps cc @JustSlone"

gouttierre avatar Dec 05 '23 12:12 gouttierre

Next steps

Dropping in an outline of the next steps for a SearchBar (or name TBD) control that combines both SearchBox and a combobox-like results dropdown.

  • [ ] Move SearchBox to stable -- this should be fine pending a final design review.
  • [ ] Combobox migrates to a hook for managing roving activedescendant values. This would involve a fairly big refactor as part of the pickers work (the current approach would be to re-use the useCombobox hook directly), so it makes sense to wait till this is done before working on combo-search. (cc/ @ling1726).
  • [ ] Final design spec review (this has already been reviewed, but should probably be looked over by whoever picks up the dev work).
    • [ ] Related: consider whether we need an abstracted layout component for the icon + text + secondary text option layout that would be reusable between Search, Combobox/Dropdown, and potentially other controls. This can also come later, and shouldn't block SearchBar.
  • [ ] Build SearchBar, using the following controls:
    • SearchBox as the input/trigger - this already has all the styles, slots and layout for the collapsed state
    • Listbox + Popover as the popup
    • The new activedescendant hook
    • Although with the proposed abstracted activedescendant approach we won't need to use Combobox directly, it'll probably still make sense to look at some of the interaction from Combobox, especially around selection and open/close behavior.

I think that's everything! The bulk of the dev work should be around hooking up the SearchBox and Listbox (and potentially some combobox hooks) so roving activedescendant and selection work. The existing components should already have all the visual styles and basic functionality covered.

smhigley avatar Jan 15 '24 16:01 smhigley

📢

  • @smhigley I see that this is ready to go to stable after a final design review. Do you have a date in mind? I believe design is happy with the spec. What are the next steps for the engineering team? Please let us know if you need any assistance. cc @behowell , @tudorpopams, @JustSlone

gouttierre avatar Feb 06 '24 13:02 gouttierre

📢

  • This week, Karina will meet up with Sarah to discuss whether the design team can support the engineering team in any way. cc @smhigley , @JustSlone , @behowell

gouttierre avatar Feb 14 '24 14:02 gouttierre

📢

  • In preview. PR made https://github.com/microsoft/fluentui/pull/30553

  • ETA for stable is E.O.M. cc @behowell , @miroslavstastny , @tudorpopams, @JustSlone

gouttierre avatar Feb 16 '24 13:02 gouttierre

  • ETA for stable is E.O.M. @gouttierre any updates on this component since this comment? Thanks.

sydsutton avatar Mar 18 '24 16:03 sydsutton

📢

  • In preview. PR in review https://github.com/microsoft/fluentui/pull/30553
  • @smhigley , @tudorpopams , @behowell, @JustSlone do you have any idea on the PR status?

gouttierre avatar Mar 20 '24 13:03 gouttierre

📢

gouttierre avatar Apr 16 '24 12:04 gouttierre

@gouttierre thanks for the update! I'm slightly confused, though, why it was moved out of Preview Components when the current version of react-components (9.47.5) doesn't contain the stable SearchBox component.

sydsutton avatar Apr 16 '24 16:04 sydsutton