Skip to content

docs(useMultipleSelection): remove itemToString prop references#1684

Merged
silviuaavram merged 2 commits into
downshift-js:masterfrom
SAY-5:docs/useMultipleSelection-remove-itemToString
May 18, 2026
Merged

docs(useMultipleSelection): remove itemToString prop references#1684
silviuaavram merged 2 commits into
downshift-js:masterfrom
SAY-5:docs/useMultipleSelection-remove-itemToString

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented May 13, 2026

Pull Request

What

Removes the itemToString prop entry and TOC link from the useMultipleSelection README. The prop was removed from the hook in v9, but the docs still describe it.

Why

Per @silviuaavram in #1655 ("Thanks!"), the docs should reflect that itemToString is no longer a useMultipleSelection prop.

How

Deleted the ### itemToString section and its TOC entry. The reference inside the getA11yStatusMessage example is preserved since the surrounding comment already notes itemToString is a user-defined function in that scope.

Changes

  • src/hooks/useMultipleSelection/README.md: remove itemToString section + TOC link.

Checklist

  • Documentation
  • Tests N/A
  • TypeScript Types N/A
  • Ready to be merged

Closes #1655

@silviuaavram
Copy link
Copy Markdown
Collaborator

Cool thanks! As a side node, we deprecated useMultipleSelection, so feel free to check useTagGroup instead in order to build multiple selection.

@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (74ce0f5) to head (d5e82d5).
⚠️ Report is 21 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##            master     #1684    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           18        54    +36     
  Lines         1726      1983   +257     
  Branches       518       604    +86     
==========================================
+ Hits          1726      1983   +257     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@silviuaavram silviuaavram merged commit cfefe77 into downshift-js:master May 18, 2026
4 checks passed
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.

Update useMultipleSelection docs and remove references to itemToString prop

3 participants