feat: enhance WelcomeBanner with settings button and update styles for better layout

This commit is contained in:
Linus Rath
2026-03-10 16:11:14 +01:00
parent c7485f532c
commit c1956bfb2d
5 changed files with 35 additions and 11 deletions
+3 -1
View File
@@ -1044,6 +1044,7 @@ export default function Home() {
)} )}
</div> </div>
<div className="flex-1 min-h-0 flex flex-col">
<WelcomeBanner /> <WelcomeBanner />
<ErrorBoundary fallback={EmailListErrorFallback}> <ErrorBoundary fallback={EmailListErrorFallback}>
@@ -1100,9 +1101,10 @@ export default function Home() {
selectEmail(email); selectEmail(email);
await handleUndoSpam(); await handleUndoSpam();
}} }}
className="flex-1" className="flex-1 min-h-0"
/> />
</ErrorBoundary> </ErrorBoundary>
</div>
</div> </div>
{/* Email list resize handle (desktop only) */} {/* Email list resize handle (desktop only) */}
+10
View File
@@ -92,6 +92,10 @@ body {
min-width: 100%; min-width: 100%;
} }
.email-content-text {
padding: 1rem 1.25rem;
}
.email-content { .email-content {
font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, 'Helvetica Neue', Arial, sans-serif; font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, 'Helvetica Neue', Arial, sans-serif;
font-size: 0.9375rem; font-size: 0.9375rem;
@@ -100,6 +104,12 @@ body {
max-width: none; max-width: none;
overflow-wrap: break-word; overflow-wrap: break-word;
word-wrap: break-word; word-wrap: break-word;
padding: 1rem 1.25rem;
}
/* Remove padding for rich HTML emails with table-based layouts */
.email-content:has(> table) {
padding: 0;
} }
.email-content p { .email-content p {
+1 -1
View File
@@ -210,7 +210,7 @@ export function EmailList({
}, [listDensity, showPreview]); }, [listDensity, showPreview]);
return ( return (
<div className={cn("flex flex-col h-full", className)}> <div className={cn("flex flex-col min-h-0", className)}>
{/* Batch Actions Toolbar */} {/* Batch Actions Toolbar */}
<div <div
className={cn( className={cn(
+20 -9
View File
@@ -2,13 +2,15 @@
import { useState, useEffect, useCallback } from "react"; import { useState, useEffect, useCallback } from "react";
import { useTranslations } from "next-intl"; import { useTranslations } from "next-intl";
import { X, Lightbulb } from "lucide-react"; import { X, Lightbulb, Settings } from "lucide-react";
import { Button } from "@/components/ui/button"; import { Button } from "@/components/ui/button";
import { useRouter } from "@/i18n/navigation";
const ONBOARDING_KEY = "onboarding_completed"; const ONBOARDING_KEY = "onboarding_completed";
export function WelcomeBanner() { export function WelcomeBanner() {
const t = useTranslations("welcome"); const t = useTranslations("welcome");
const router = useRouter();
const [visible, setVisible] = useState(false); const [visible, setVisible] = useState(false);
const [dismissed, setDismissed] = useState(false); const [dismissed, setDismissed] = useState(false);
@@ -42,24 +44,24 @@ export function WelcomeBanner() {
<div <div
role="complementary" role="complementary"
aria-label={t("title")} aria-label={t("title")}
className={`mx-4 mt-3 mb-1 rounded-lg border border-border bg-background shadow-sm transition-all duration-300 ease-out ${ className={`border-b border-border bg-accent/30 transition-all duration-300 ease-out ${
dismissed ? "opacity-0 scale-95 pointer-events-none" : "opacity-100 scale-100" dismissed ? "opacity-0 scale-y-0 max-h-0 pointer-events-none" : "opacity-100 scale-y-100 max-h-96"
}`} }`}
onTransitionEnd={() => { onTransitionEnd={() => {
if (dismissed) setVisible(false); if (dismissed) setVisible(false);
}} }}
> >
<div className="p-4"> <div className="px-4 py-3">
<div className="flex items-start justify-between gap-3"> <div className="flex items-start justify-between gap-3">
<div className="flex items-start gap-3"> <div className="flex items-start gap-3">
<div className="flex-shrink-0 mt-0.5 p-1.5 rounded-md bg-primary/10"> <div className="flex-shrink-0 mt-0.5 p-1.5 rounded-md bg-primary/10">
<Lightbulb className="w-4 h-4 text-primary" /> <Lightbulb className="w-4 h-4 text-primary" />
</div> </div>
<div className="space-y-2"> <div className="space-y-1.5">
<h3 className="text-sm font-medium text-foreground"> <h3 className="text-sm font-medium text-foreground">
{t("title")} {t("title")}
</h3> </h3>
<ul className="space-y-1.5 text-sm text-muted-foreground"> <ul className="space-y-1 text-sm text-muted-foreground">
<li>{t("tip_compose")}</li> <li>{t("tip_compose")}</li>
<li>{t("tip_shortcuts")}</li> <li>{t("tip_shortcuts")}</li>
<li>{t("tip_sidebar")}</li> <li>{t("tip_sidebar")}</li>
@@ -75,12 +77,21 @@ export function WelcomeBanner() {
<X className="w-4 h-4" /> <X className="w-4 h-4" />
</button> </button>
</div> </div>
<div className="mt-3 flex justify-end"> <div className="mt-2.5 flex justify-end gap-2">
<Button <Button
variant="ghost" variant="outline"
size="sm"
onClick={() => { dismiss(); router.push('/settings'); }}
className="text-xs h-7"
>
<Settings className="w-3.5 h-3.5 mr-1" />
{t("settings")}
</Button>
<Button
variant="default"
size="sm" size="sm"
onClick={dismiss} onClick={dismiss}
className="text-xs" className="text-xs h-7"
> >
{t("got_it")} {t("got_it")}
</Button> </Button>
+1
View File
@@ -1426,6 +1426,7 @@
"tip_sidebar": "Find Calendar, Contacts and Settings in the sidebar menu", "tip_sidebar": "Find Calendar, Contacts and Settings in the sidebar menu",
"tip_settings": "Customize your experience in Settings", "tip_settings": "Customize your experience in Settings",
"got_it": "Got it", "got_it": "Got it",
"settings": "Settings",
"dismiss": "Dismiss" "dismiss": "Dismiss"
} }
} }