Skip to content

Conversation

@reneshen0328
Copy link
Contributor

@reneshen0328 reneshen0328 commented Feb 10, 2026

Summary

Bump @box/unified-share-modal and related dependencies to support the desktop view variant for the Content Sharing element.

Changes

Dependency Updates:

Package Before After
@box/unified-share-modal 1.37.1 1.45.0
@box/blueprint-web 12.130.0 12.132.0
@box/blueprint-web-assets 4.97.3 4.99.5
@box/copy-input 1.22.3 1.27.4
@box/user-selector 1.55.3 1.60.4

New Feature:

  • Added variant prop to ContentSharing and ContentSharingV2 components
  • Supports "desktop" and "modal" variants of the Unified Share Modal

Summary by CodeRabbit

  • New Features

    • Added optional variant support to content sharing interface, allowing selection between desktop and modal display modes for improved flexibility.
  • Chores

    • Updated UI framework dependencies (blueprint-web, blueprint-web-assets, copy-input, unified-share-modal, and user-selector) to latest compatible versions for enhanced stability and performance.

@reneshen0328 reneshen0328 requested review from a team as code owners February 10, 2026 17:23
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 10, 2026

Walkthrough

Updates five Box UI library dependencies across devDependencies and peerDependencies in package.json. Adds a new optional variant prop to ContentSharing and ContentSharingV2 components to support different rendering modes, with the prop flowing from ContentSharing through to UnifiedShareModal. Includes corresponding story updates to demonstrate the new variant options.

Changes

Cohort / File(s) Summary
Dependency Updates
package.json
Updated versions for @box/blueprint-web (12.130.0→12.132.0), @box/blueprint-web-assets (4.97.3→4.99.5), @box/copy-input (1.22.3→1.27.4), @box/unified-share-modal (1.37.1→1.45.0), and @box/user-selector (1.55.3→1.60.4) across devDependencies and peerDependencies.
Component Prop Addition
src/elements/content-sharing/ContentSharing.js, src/elements/content-sharing/ContentSharingV2.tsx
Added optional variant?: VariantType prop to both components. Imported VariantType from @box/unified-share-modal and forwarded the variant prop through the component hierarchy.
Story Updates
src/elements/content-sharing/stories/ContentSharing.stories.js, src/elements/content-sharing/stories/ContentSharingV2.stories.tsx, src/elements/content-sharing/stories/tests/ContentSharingV2-visual.stories.tsx
Added variant demonstration to stories: modal variant to ContentSharing story args, new desktopVariant export with variant='desktop' to V2 story, and DesktopVariant visual test story with Close button absence assertion.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • jfox-box
  • tjuanitas

Poem

🐰 A variant hops through the code so neat,
Desktop and modal, a sharing feat!
Props cascade down like carrots in spring,
Stories now show what the changes bring! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The PR title accurately describes the main change—bumping unified-share-modal version to enable desktop view support, which is the core objective.
Description check ✅ Passed The PR description provides a clear summary, comprehensive dependency update table, and documents the new variant prop feature, covering the main changes effectively.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat-bump-content-sharing-usm

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
src/elements/content-sharing/stories/tests/ContentSharingV2-visual.stories.tsx (1)

47-56: Naming inconsistency: DesktopVariant uses PascalCase while sibling exports use camelCase.

All other story exports in this file (withModernization, withSharedLink, withCollaborators) use camelCase. Consider renaming to desktopVariant for consistency.

Proposed rename
-export const DesktopVariant = {
+export const desktopVariant = {

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mergify mergify bot added the queued label Feb 10, 2026
@mergify mergify bot merged commit 8c84972 into master Feb 10, 2026
12 checks passed
@mergify mergify bot deleted the feat-bump-content-sharing-usm branch February 10, 2026 17:45
@mergify
Copy link
Contributor

mergify bot commented Feb 10, 2026

Merge Queue Status

Rule: Automatic strict merge


  • Entered queue2026-02-10 17:45 UTC
  • Checks passed · in-place
  • Merged2026-02-10 17:45 UTC · at b0648e21db3fcd0cb37338ea381f073919ad7321

This pull request spent 17 seconds in the queue, including 1 second running CI.

Required conditions to merge

@mergify mergify bot removed the queued label Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants