Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added feature to add action button to terms in ModernTaxonomyPicker #1058

Merged
merged 6 commits into from
Dec 21, 2021

Conversation

patrikhellgren
Copy link
Contributor

@patrikhellgren patrikhellgren commented Nov 10, 2021

Q A
Bug fix? [ ]
New feature? [X]
New sample? [ ]
Related issues?

What's in this Pull Request?

Added a feature to ModernTaxonomyPicker to add a custom action button to each term in the term set and the term set itself.
This also comes with the added bonus of better handling of keyboard navigation.
Added properties isLightDismiss and isBlocking that will be used for the panel.
Refactored TaxonomyTree into a separate component that could be used separately to just display a term set with action buttons for e.g. an admin UI.

Here is an example of what it could look like with a context menu for adding and deleting terms in the control:

image

@estruyf estruyf deleted the branch pnp:dev December 3, 2021 08:27
@estruyf estruyf closed this Dec 3, 2021
@patrikhellgren
Copy link
Contributor Author

Hi @estruyf, What happened to this PR, why was it closed without being merged first?

@estruyf
Copy link
Member

estruyf commented Dec 3, 2021

@patrikhellgren I have no idea why the PR got closed. Going to re-open it.

@estruyf estruyf reopened this Dec 3, 2021
@AJIXuMuK AJIXuMuK merged commit ee42842 into pnp:dev Dec 21, 2021
@AJIXuMuK
Copy link
Collaborator

Thank you @patrikhellgren for these updates!

@AJIXuMuK AJIXuMuK added this to the 3.6.0 milestone Dec 21, 2021
@patrikhellgren patrikhellgren deleted the modern-taxonomy-picker-2 branch January 5, 2022 14:53
@AJIXuMuK AJIXuMuK mentioned this pull request Feb 12, 2022
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.

None yet

3 participants