Skip to content

Commit 2f70ec8

Browse files
author
Ritika Mishra
committed
add hidden dialog title for contributors modal and add toast for zip downloaded
1 parent 15dca89 commit 2f70ec8

File tree

2 files changed

+15
-12
lines changed

2 files changed

+15
-12
lines changed

src/components/Connection.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -427,8 +427,10 @@ const Connection: React.FC<ConnectionProps> = ({
427427

428428
if (zipBlob) {
429429
saveAs(zipBlob, 'ChordsWeb.zip');
430+
toast.success("Data successfully downloaded as ZIP.");
430431
} else if (error) {
431432
console.error(error);
433+
toast.error(`Error while creating ZIP: ${error}`);
432434
}
433435
};
434436
}

src/components/Contributors.tsx

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import {
1313
} from "../components/ui/card";
1414
import { Separator } from "../components/ui/separator";
1515
import { Avatar, AvatarFallback, AvatarImage } from "../components/ui/avatar";
16-
import { Dialog, DialogContent, DialogTrigger } from "../components/ui/dialog";
16+
import { Dialog, DialogContent, DialogTrigger, DialogTitle } from "../components/ui/dialog";
1717
import { CircleAlert } from "lucide-react";
1818
import { Button } from "../components/ui/button";
1919
import Link from "next/link";
@@ -62,22 +62,23 @@ const contributors = [
6262
export default function Contributors() {
6363
return (
6464
<Dialog>
65-
<DialogTrigger asChild>
66-
<TooltipProvider>
67-
<Tooltip>
68-
<TooltipTrigger asChild>
65+
<TooltipProvider>
66+
<Tooltip>
67+
<TooltipTrigger asChild>
68+
<DialogTrigger asChild>
6969
<Button variant="ghost" size="icon">
7070
<CircleAlert className="h-5 w-5" />
7171
<span className="sr-only">View Contributors</span>
7272
</Button>
73-
</TooltipTrigger>
74-
<TooltipContent>
75-
<p>Contributors</p>
76-
</TooltipContent>
77-
</Tooltip>
78-
</TooltipProvider>
79-
</DialogTrigger>
73+
</DialogTrigger>
74+
</TooltipTrigger>
75+
<TooltipContent>
76+
<p>Contributors</p>
77+
</TooltipContent>
78+
</Tooltip>
79+
</TooltipProvider>
8080
<DialogContent className="sm:max-w-[425px] md:max-w-[570px] lg:max-w-[650px]">
81+
<DialogTitle className="sr-only">Contributors</DialogTitle>
8182
<Card className="border-none">
8283
<CardHeader className="p-0 mb-2">
8384
<CardTitle className="font-bold items-center gap-2 flex mb-1">

0 commit comments

Comments
 (0)