Skip to content

ref(issues): Remove some usages of useHasStreamlinedUI#111036

Merged
malwilley merged 2 commits intomasterfrom
malwilley/remove-streamlined-ui
Mar 18, 2026
Merged

ref(issues): Remove some usages of useHasStreamlinedUI#111036
malwilley merged 2 commits intomasterfrom
malwilley/remove-streamlined-ui

Conversation

@malwilley
Copy link
Member

One part of a many code deletions now that the old experience is dead.

This hits a few misc components that rendered differently in the old UI.

@malwilley malwilley requested review from a team as code owners March 18, 2026 21:55
@malwilley malwilley removed the request for review from a team March 18, 2026 21:55
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 18, 2026
@malwilley malwilley requested a review from scttcper March 18, 2026 21:55

export const TreeContainer = styled('div')<{columnCount: number}>`
margin-top: ${p => p.theme.space.lg};
margin-top: 0;
Copy link
Member

Choose a reason for hiding this comment

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

might be able to remove now

Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Required size prop is accepted but silently unused
    • Removed the unused size: number from the type definition and removed the size prop from both usages in avatarList.tsx and editAccessSelector.tsx.

Create PR

Or push these changes by commenting:

@cursor push 0352bde6d3
Preview (0352bde6d3)
diff --git a/static/app/components/core/avatar/avatarList.tsx b/static/app/components/core/avatar/avatarList.tsx
--- a/static/app/components/core/avatar/avatarList.tsx
+++ b/static/app/components/core/avatar/avatarList.tsx
@@ -32,7 +32,6 @@
   children,
 }: {
   children: React.ReactNode;
-  size: number;
   ref?: React.Ref<HTMLDivElement>;
 }) {
   return (
@@ -85,10 +84,7 @@
           renderCollapsedAvatars(avatarSize, numCollapsedAvatars)
         ) : (
           <Tooltip title={`${numCollapsedAvatars} other ${typeAvatars}`} skipWrapper>
-            <CollapsedAvatars
-              size={avatarSize}
-              data-test-id="avatarList-collapsedavatars"
-            >
+            <CollapsedAvatars data-test-id="avatarList-collapsedavatars">
               {numCollapsedAvatars < 99 && '+'}
               {numCollapsedAvatars}
             </CollapsedAvatars>

diff --git a/static/app/views/dashboards/editAccessSelector.tsx b/static/app/views/dashboards/editAccessSelector.tsx
--- a/static/app/views/dashboards/editAccessSelector.tsx
+++ b/static/app/views/dashboards/editAccessSelector.tsx
@@ -180,7 +180,7 @@
           onMouseEnter={() => setIsCollapsedAvatarTooltipOpen(true)}
           onMouseLeave={() => setIsCollapsedAvatarTooltipOpen(false)}
         >
-          <CollapsedAvatars size={avatarSize}>
+          <CollapsedAvatars>
             {numCollapsedAvatars < 99 && <Plus>+</Plus>}
             {numCollapsedAvatars}
           </CollapsedAvatars>

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

@malwilley malwilley requested review from a team as code owners March 18, 2026 22:28
@malwilley malwilley merged commit 7e0f975 into master Mar 18, 2026
64 checks passed
@malwilley malwilley deleted the malwilley/remove-streamlined-ui branch March 18, 2026 23:13
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.

2 participants