Skip to content

Replace removed Box component with div for @primer/react v38 compatibility#1596

Closed
Copilot wants to merge 3 commits intorenovate/major-primerfrom
copilot/sub-pr-1595
Closed

Replace removed Box component with div for @primer/react v38 compatibility#1596
Copilot wants to merge 3 commits intorenovate/major-primerfrom
copilot/sub-pr-1595

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 19, 2026

Box was removed from @primer/react v38, breaking the Vite build. Replace the single usage in notebook_url_share_overlay.tsx with a native div — functionally equivalent since Box was only used as a className-bearing container.

// Before
import { Box } from '@primer/react';
<Box className={classNames(styles.sharing_overlay, props.className)}>

// After
<div className={classNames(styles.sharing_overlay, props.className)}>

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Co-authored-by: ankoh <3986510+ankoh@users.noreply.github.com>
Copilot AI changed the title [WIP] Update dependency @primer/react to v38 Replace removed Box component with div for @primer/react v38 compatibility Mar 19, 2026
Copilot AI requested a review from ankoh March 19, 2026 07:59
@dashql-renovate dashql-renovate bot force-pushed the renovate/major-primer branch 4 times, most recently from de6437e to 5927b59 Compare March 23, 2026 09:23
@dashql-renovate dashql-renovate bot force-pushed the renovate/major-primer branch 4 times, most recently from 36dd24b to 4fac6a7 Compare April 2, 2026 21:12
@ankoh ankoh closed this Apr 7, 2026
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.

2 participants