Skip to content

Commit 28ac2bf

Browse files
tune email verification dialog copy and affected-user header
Use singular/plural description text; move the total count next to the affected accounts label; shorten the list footer to “+ N more”. Made-with: Cursor
1 parent 7112f49 commit 28ac2bf

1 file changed

Lines changed: 17 additions & 19 deletions

File tree

  • apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/onboarding

apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/onboarding/page-client.tsx

Lines changed: 17 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,13 @@ function EnableEmailVerificationDialog({
3737
pendingChange: PendingChange | null,
3838
onDismiss: () => void,
3939
}) {
40+
const description =
41+
pendingChange == null
42+
? undefined
43+
: pendingChange.totalAffectedCount === 1
44+
? "Existing user who has not verified will need to complete verification the next time they open your app."
45+
: "Existing users who have not verified will need to complete verification the next time they open your app.";
46+
4047
return (
4148
<DesignDialog
4249
open={pendingChange != null}
@@ -48,21 +55,7 @@ function EnableEmailVerificationDialog({
4855
size="lg"
4956
icon={WarningCircle}
5057
title="Enable email verification?"
51-
description="Existing users who have not verified will need to complete verification the next time they open your app."
52-
headerContent={
53-
pendingChange != null && pendingChange.totalAffectedCount > 0 ? (
54-
<p className="text-sm text-muted-foreground">
55-
<span className="font-semibold tabular-nums text-foreground">
56-
{pendingChange.totalAffectedCount.toLocaleString()}
57-
</span>
58-
{" "}
59-
user
60-
{pendingChange.totalAffectedCount === 1 ? "" : "s"}
61-
{" "}
62-
may be asked to verify on their next app open. The list below is a sample; totals may be higher.
63-
</p>
64-
) : null
65-
}
58+
description={description}
6659
footer={(
6760
<>
6861
<DesignDialogClose asChild>
@@ -86,9 +79,14 @@ function EnableEmailVerificationDialog({
8679
<div className="flex flex-col gap-3">
8780
{pendingChange.affectedUsers.length > 0 && (
8881
<div>
89-
<Typography variant="secondary" className="mb-2 text-[10px] font-semibold uppercase tracking-wider">
90-
Sample accounts
91-
</Typography>
82+
<div className="mb-2 flex flex-wrap items-baseline gap-2">
83+
<span className="text-lg font-semibold tabular-nums text-primary">
84+
{pendingChange.totalAffectedCount.toLocaleString()}
85+
</span>
86+
<Typography variant="secondary" className="text-[10px] font-semibold uppercase tracking-wider">
87+
Affected accounts
88+
</Typography>
89+
</div>
9290
<div className="max-h-[min(200px,35vh)] overflow-y-auto rounded-xl bg-background/60 ring-1 ring-foreground/[0.06]">
9391
<ul className="divide-y divide-foreground/[0.06]">
9492
{pendingChange.affectedUsers.map((user) => (
@@ -122,7 +120,7 @@ function EnableEmailVerificationDialog({
122120
{" "}
123121
{(pendingChange.totalAffectedCount - pendingChange.affectedUsers.length).toLocaleString()}
124122
{" "}
125-
more not shown in this sample
123+
more
126124
</p>
127125
)}
128126
</div>

0 commit comments

Comments
 (0)