From bef6b4c30bd5c51dd69e4308690915cc07e91e1f Mon Sep 17 00:00:00 2001 From: Jake Howell Date: Tue, 25 Aug 2026 14:07:52 +0700 Subject: [PATCH] revert(site): remove built-in emoji avatar inset (#28533) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit > 🤖 This PR was written by Coder Agents on behalf of Jake Howell. Reverts `75400428ad2b84c707f60fc5e6b378f46f565ef9` (`fix(site/src): give built-in emoji avatars a consistent inset`). The global built-in emoji detection and inline padding are causing regressions. Revert this behavior so a replacement can be developed separately. ## Changes - Remove automatic `/emojis/` source detection from `Avatar`. - Restore caller-controlled avatar variants and styles. - Remove the built-in emoji inset stories. ## Verification - Confirmed the resulting patch is the inverse of the target commit. --- site/src/components/Avatar/Avatar.stories.tsx | 20 ------------------- site/src/components/Avatar/Avatar.tsx | 15 +------------- 2 files changed, 1 insertion(+), 34 deletions(-) diff --git a/site/src/components/Avatar/Avatar.stories.tsx b/site/src/components/Avatar/Avatar.stories.tsx index a9c51fef621..4b6b020dd8a 100644 --- a/site/src/components/Avatar/Avatar.stories.tsx +++ b/site/src/components/Avatar/Avatar.stories.tsx @@ -53,26 +53,6 @@ export const NonSquaredIcon: Story = { }, }; -export const BuiltInEmojiLgSize: Story = { - args: { - size: "lg", - src: "/emojis/1f64c.png", - }, -}; - -export const BuiltInEmojiMdSize: Story = { - args: { - src: "/emojis/1f64c.png", - }, -}; - -export const BuiltInEmojiSmSize: Story = { - args: { - size: "sm", - src: "/emojis/1f64c.png", - }, -}; - export const FallbackLgSize: Story = { args: { src: "", diff --git a/site/src/components/Avatar/Avatar.tsx b/site/src/components/Avatar/Avatar.tsx index 71dd67b05de..d080ebed312 100644 --- a/site/src/components/Avatar/Avatar.tsx +++ b/site/src/components/Avatar/Avatar.tsx @@ -73,26 +73,13 @@ export const Avatar: React.FC = ({ fallback, alt = "", children, - style, ...props }) => { const { externalImages } = useAppearance(); - const isEmoji = src?.startsWith("/emojis/"); - return (