Skip to content

Commit 9104d1a

Browse files
committed
Fix Logo Swap UI issues
- Scale logo containers to 256px (w-64) for better visibility - Add gap-8 between logos and separator for better spacing - Fix background section empty state copy for logo-swap format - Add shrink-0 to separator to prevent flex compression
1 parent e2540cb commit 9104d1a

2 files changed

Lines changed: 11 additions & 5 deletions

File tree

apps/app/src/components/layouts/LogoSwap.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ function LogoSwapComponent({
8686
if (separatorStyle === "none") return null;
8787

8888
return (
89-
<div className="flex items-center justify-center px-4">
89+
<div className="flex shrink-0 items-center justify-center">
9090
{separatorStyle === "dash" && (
9191
<div className="h-[2px] w-10 rounded-full bg-white/25" />
9292
)}
@@ -153,16 +153,16 @@ function LogoSwapComponent({
153153
data-export-element
154154
data-element="container"
155155
>
156-
<div className="flex items-center">
156+
<div className="flex items-center gap-8">
157157
{/* Left logo */}
158-
<div className="flex h-40 w-40 items-center justify-center">
158+
<div className="flex h-64 w-64 items-center justify-center">
159159
{renderLogo(leftLogoAsset, "left")}
160160
</div>
161161

162162
{renderSeparator()}
163163

164164
{/* Right logo */}
165-
<div className="flex h-40 w-40 items-center justify-center">
165+
<div className="flex h-64 w-64 items-center justify-center">
166166
{renderLogo(rightLogoAsset, "right")}
167167
</div>
168168
</div>

apps/app/src/components/sidebar/background-section.tsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import { track } from "@/lib/analytics";
2626
import type { BackgroundConfig, ColorToken } from "@/domain/layout/types";
2727
import type { BackgroundSelection, PersonalBackground } from "@/domain/backgrounds/types";
2828
import type { Asset } from "@/domain/asset/types";
29+
import { getLayoutFormat } from "@/domain/layout-def/definitions";
2930

3031
interface BackgroundSectionProps {
3132
variant?: "default" | "inline";
@@ -204,10 +205,15 @@ export function BackgroundSection({ variant = "default" }: BackgroundSectionProp
204205
[totalPages],
205206
);
206207

208+
const isLogoSwapFormat = getLayoutFormat(config.layoutId) === "logo-swap";
209+
const emptyStateMessage = isLogoSwapFormat
210+
? "Upload logos to generate backgrounds."
211+
: "Upload a screenshot to generate backgrounds.";
212+
207213
const emptyState = (
208214
<div className="rounded-lg border border-border/60 bg-muted/30 px-3 py-3">
209215
<div className="text-xs text-muted-foreground">
210-
Upload a screenshot to generate backgrounds.
216+
{emptyStateMessage}
211217
</div>
212218
</div>
213219
);

0 commit comments

Comments
 (0)