Skip to content

Commit 621acef

Browse files
authored
feat: improve tooltip language in public form header (#1334)
* feat: improve tooltip language in public form header * fix form test
1 parent ccb298c commit 621acef

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

core/app/components/pubs/PubEditor/SaveStatus.tsx

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -32,26 +32,26 @@ export const useSaveStatus = ({ defaultMessage }: { defaultMessage?: string }) =
3232

3333
export const SaveStatus = ({ autosave }: { autosave?: boolean }) => {
3434
const defaultMessage = autosave
35-
? "Progress will be automatically saved"
35+
? "Form will save every few seconds while editing"
3636
: "Form will save when you click submit";
3737
const status = useSaveStatus({ defaultMessage });
3838

3939
return (
4040
<div className="flex items-center gap-2 text-sm font-normal text-muted-foreground">
4141
<span>{status}</span>
42-
{autosave ? (
43-
<Tooltip>
44-
<TooltipContent side="bottom" className="w-60 text-sm">
45-
Bookmark this page to return to your saved progress anytime.
46-
</TooltipContent>
47-
<TooltipTrigger asChild>
48-
<div>
49-
<span className="sr-only">More info</span>
50-
<CircleHelp size={16} />
51-
</div>
52-
</TooltipTrigger>
53-
</Tooltip>
54-
) : null}
42+
<Tooltip>
43+
<TooltipContent side="bottom" className="w-60 text-sm">
44+
{autosave
45+
? "Progress is automatically saved every few seconds while editing. Bookmark this page to return to your saved progress anytime."
46+
: "Progress made on this form is not automatically saved. You will lose unsaved changes if you navigate away from this page."}
47+
</TooltipContent>
48+
<TooltipTrigger asChild>
49+
<div>
50+
<span className="sr-only">More info</span>
51+
<CircleHelp size={16} />
52+
</div>
53+
</TooltipTrigger>
54+
</Tooltip>
5555
</div>
5656
);
5757
};

core/playwright/externalFormInvite.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@ test.describe("Inviting a new user to fill out a form", () => {
251251
// Open a new page so that we're no longer logged in as admin
252252
const newPage = await browser.newPage();
253253
await newPage.goto(decodedUrl);
254-
await newPage.getByText("Progress will be automatically saved").waitFor();
254+
await newPage.getByText("Form will save every few seconds while editing").waitFor();
255255

256256
await newPage.getByLabel("Content").fill("LGTM");
257257

0 commit comments

Comments
 (0)