Skip to content
This repository was archived by the owner on Jun 8, 2026. It is now read-only.

Commit af76dfd

Browse files
mikeangstadtclaude
andcommitted
FEA-1550: Fix sidebar icon sizing and CSS import order
- Import design-system globals.css before local globals.css so DS styles are the base layer and local overrides apply on top - Replace arbitrary size-[18px] span wrapper with standard size-4 class directly on SVG elements, matching the design system's own icon sizing convention and ensuring Tailwind generates the class Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent edf1b31 commit af76dfd

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

apps/desktop/src/renderer/components/layout/Sidebar.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ const SVG_ICONS: Record<string, string> = {
4848
function NavIcon({ name }: { name: string }) {
4949
const paths = SVG_ICONS[name] || "";
5050
return (
51-
<span className="shrink-0 size-[18px] text-current opacity-70">
52-
<svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" className="size-full">
51+
<span className="shrink-0 text-current opacity-70">
52+
<svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" className="size-4">
5353
<g dangerouslySetInnerHTML={{ __html: paths }} />
5454
</svg>
5555
</span>

apps/desktop/src/renderer/main.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import { StrictMode } from "react";
22
import { createRoot } from "react-dom/client";
33
import { DesignSystemProvider } from "@closedloop-ai/design-system";
4-
import "./globals.css";
54
import "@closedloop-ai/design-system/styles/globals.css";
5+
import "./globals.css";
66
import App from "./App";
77

88
window.addEventListener("error", (event) => {

0 commit comments

Comments
 (0)