Skip to content

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Dec 21, 2025

Summary

This PR removes unused withChonk imports and related unused code from shared components across the application.

Changes

  • Removed unused withChonk imports from 20 component files
  • Removed unused helper functions and styled components that are no longer needed
  • Cleaned up unused imports (css, styled, Theme, etc.) that were only used by the removed code

Files affected

  • static/app/components/actions/resolve.tsx
  • static/app/components/assigneeBadge.tsx
  • static/app/components/events/eventDrawer.tsx
  • static/app/components/events/eventReplay/replayPreviewPlayer.tsx
  • static/app/components/globalModal/components.tsx
  • static/app/components/onboarding/productSelection.tsx
  • static/app/components/overlay.tsx
  • static/app/components/overlayArrow.tsx
  • static/app/components/profiling/flamegraph/collapsibleTimeline.tsx
  • static/app/components/profiling/flamegraph/flamegraphDrawer/flamegraphDrawer.tsx
  • static/app/views/explore/logs/styles.tsx
  • static/app/views/explore/spans/spansTab.tsx
  • static/app/views/insights/aiGenerations/views/overview.tsx
  • static/app/views/issueDetails/streamline/eventGraph.tsx
  • static/app/views/issueDetails/streamline/groupDetailsLayout.tsx
  • static/app/views/issueList/issueViews/issueViewSaveButton.tsx
  • static/app/views/nav/primary/components.tsx
  • static/app/views/nav/secondary/secondary.tsx
  • static/app/views/nav/sidebar.tsx
  • static/app/views/settings/components/settingsBreadcrumb/divider.tsx

This is part of a larger effort to clean up unused code following the chonk migration.

Ref DE-498

@JonasBa JonasBa requested review from a team as code owners December 21, 2025 22:52
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 21, 2025
@linear
Copy link

linear bot commented Dec 21, 2025

@JonasBa JonasBa requested a review from a team December 22, 2025 19:34
Copy link
Member

@natemoo-re natemoo-re left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything here looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants