From 62b132b6984797d7b66ac0023ad397ad5f66d3a4 Mon Sep 17 00:00:00 2001
From: Xubin Ren <52506698+Re-bin@users.noreply.github.com>
Date: Tue, 28 Jul 2026 17:52:42 +0800
Subject: [PATCH] style(webui): reduce repeated marketplace labels

---
 webui/src/components/settings/SkillsMarketplace.tsx | 10 +---------
 webui/src/tests/app-layout.test.tsx                 |  4 ++--
 2 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/webui/src/components/settings/SkillsMarketplace.tsx b/webui/src/components/settings/SkillsMarketplace.tsx
index 5ab84a174..0b62acb86 100644
--- a/webui/src/components/settings/SkillsMarketplace.tsx
+++ b/webui/src/components/settings/SkillsMarketplace.tsx
@@ -504,7 +504,6 @@ function MarketplaceSkillRow({
           </a>
         </div>
         <div className="mt-1 flex min-w-0 items-center gap-1.5 truncate text-[12px] text-muted-foreground">
-          <ProviderMark provider={skill.provider} compact />
           {skill.source}
           {skill.version ? <span>· v{skill.version}</span> : null}
           <span>·</span>
@@ -572,18 +571,11 @@ function MarketplaceSkillRow({
 
 function ProviderMark({
   provider,
-  compact = false,
 }: {
   provider: Exclude<MarketplaceProvider, "all">;
-  compact?: boolean;
 }) {
   return (
-    <span
-      className={cn(
-        "inline-flex shrink-0 items-center gap-1.5 font-medium",
-        compact ? "text-[11px] text-muted-foreground" : "text-[12px] text-foreground/75",
-      )}
-    >
+    <span className="inline-flex shrink-0 items-center gap-1.5 text-[12px] font-medium text-foreground/75">
       <ProviderDot provider={provider} />
       {providerLabel(provider)}
     </span>
diff --git a/webui/src/tests/app-layout.test.tsx b/webui/src/tests/app-layout.test.tsx
index 4afec428f..c2d1b1a9a 100644
--- a/webui/src/tests/app-layout.test.tsx
+++ b/webui/src/tests/app-layout.test.tsx
@@ -714,8 +714,8 @@ describe("App layout", () => {
     ).toBeInTheDocument();
     expect(screen.getByText("find-skills")).toBeInTheDocument();
     expect(screen.getByText("ima-skills")).toBeInTheDocument();
-    expect(screen.getAllByText("SkillHub").length).toBeGreaterThan(0);
-    expect(screen.getAllByText("skills.sh").length).toBeGreaterThan(0);
+    expect(screen.getAllByText("SkillHub")).toHaveLength(2);
+    expect(screen.getAllByText("skills.sh")).toHaveLength(2);
     expect(screen.getByText(/14,481 installs \/ 24h/)).toBeInTheDocument();
     expect(
       await screen.findByRole("img", { name: "8-week install trend" }),