Skip to content

fix: makes productIcon optional in ProductSwitcherItem #4359

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

Merged

Conversation

simpixelated
Copy link
Contributor

@simpixelated simpixelated commented Jul 15, 2025

Twilio teams (Twilio, Segment, and SendGrid) are updating the design of the unified identity product switcher. Currently, we're hacking in a solution by setting productIcon={<></>} or similar and passing custom styles to adjust the padding. This works for now, but ideally we should make this property optional for this new use-case.

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

changeset-bot bot commented Jul 15, 2025

🦋 Changeset detected

Latest commit: 8def605

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@twilio-paste/product-switcher Patch
@twilio-paste/core Patch
@twilio-paste/website Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Jul 15, 2025

@simpixelated is attempting to deploy a commit to the Twilio Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

codesandbox-ci bot commented Jul 15, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8def605:

Sandbox Source
@twilio-paste/nextjs-template Configuration
@twilio-paste/token-contrast-checker Configuration

simpixelated and others added 2 commits July 16, 2025 13:14
Co-authored-by: krisantrobus <55083528+krisantrobus@users.noreply.github.com>
@simpixelated simpixelated marked this pull request as ready for review July 16, 2025 17:41
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 16, 2025
Copy link

vercel bot commented Jul 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
paste-docs ✅ Ready (Inspect) Visit Preview Jul 16, 2025 8:17pm
paste-remix ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 16, 2025 8:17pm

serifluous
serifluous previously approved these changes Jul 16, 2025
@serifluous serifluous added the 🕵🏻‍♀️ Run website visual regression When applied, we will run a full suite of visual regression tests across the doc site label Jul 16, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 16, 2025
@krisantrobus krisantrobus added the Contribution This is a contribution label Jul 17, 2025
@krisantrobus krisantrobus merged commit 637362e into twilio-labs:main Jul 18, 2025
13 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Components Related to the component library (core) of this system Contribution This is a contribution lgtm This PR has been approved by a maintainer 🕵🏻‍♀️ Run website visual regression When applied, we will run a full suite of visual regression tests across the doc site size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants