Skip to content

Conversation

@emplums
Copy link

@emplums emplums commented Mar 5, 2020

This PR adds the ability to pass a React node to the title prop of the Dropdown component. This will allow users to add Octicons to the title.

Closes: #699

Merge checklist

  • Added or updated TypeScript definitions (index.d.ts) if necessary
  • Added/updated tests
  • Added/updated documentation
  • Tested in Chrome
  • Tested in Firefox
  • Tested in Safari
  • Tested in Edge

Take a look at the What we look for in reviews section of the contibuting guidelines for more infomation on how we review PRs.

@vercel
Copy link

vercel bot commented Mar 5, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/primer/primer-components/1uzhz7fa8
✅ Preview: https://primer-components-git-dropdown-type.primer.now.sh

@emplums emplums requested a review from dmarcey March 5, 2020 21:00
@emplums emplums added the minor release new features label Mar 5, 2020
@emplums emplums changed the title Fix: update Dropdown title type Update Dropdown title type Mar 5, 2020
Copy link
Contributor

@dmarcey dmarcey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 types!

@emplums emplums changed the base branch from master to minor March 17, 2020 00:06
@emplums emplums merged commit fe77946 into minor Mar 17, 2020
@emplums emplums deleted the dropdown-type branch September 16, 2020 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor release new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[API] Dropdown button contents - TypeScript

3 participants