[Feature Request]: Disable Modal for TagSet component

jeesonjohnson commented 2 months ago

The problem

Currently, we do not have the option to disable the TagSet overflow modal. This is currently an issue for our product, since we chose to disable this component, in favour of our own custom component with custom functionality. However, with how the code is set up currently, the inbuilt TagSetModal cannot be disabled. See here. We have disabled it manually using CSS overrides. However, we would rather it be disabled by the use of a prop, so that the DOM does not become overly populated unnecessarily.

The solution

Pass down a prop called, showOverflowModal (defaulted to true) that will disable the rendering of the component within the TagSet component.

NOTE: I am happy to do this change, raise the appropriate PRs and do related testing. Just wanted to approval before attempting to make this change.

Design link

N/A

Links to other materials

Owner/main maintainer(s)

IBM Products

Second/backup maintainer(s)

No response

Product/offering

IBM Cloud Pak For Integration

Business priority

Low Priority = release date is not dependent on fix or not upcoming

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

oliviaflory commented 1 month ago

@matthewgallo following up with Michael G

jeesonjohnson commented 1 month ago

Hey @matthewgallo / @oliviaflory, has there been any update on this?

carbon-design-system / ibm-products