From ccfff6b1954030a273b0612f3772ec00a82422c8 Mon Sep 17 00:00:00 2001
From: Md Saban <45597394+mdsaban@users.noreply.github.com>
Date: Sat, 22 Jun 2024 21:20:14 +0530
Subject: fix(web): Fix save action on empty card. Fixes #225 (#243)

* fix: Empty item save action

* fix: resolve comments

* chore: prettier
---
 .../components/dashboard/bookmarks/EditorCard.tsx  | 11 ++++++----
 apps/web/components/ui/info-tooltip.tsx            |  6 +++++-
 apps/web/lib/utils.ts                              | 24 ++++++++++++++++++++++
 3 files changed, 36 insertions(+), 5 deletions(-)

(limited to 'apps/web')

diff --git a/apps/web/components/dashboard/bookmarks/EditorCard.tsx b/apps/web/components/dashboard/bookmarks/EditorCard.tsx
index 3006a964..a1055e8e 100644
--- a/apps/web/components/dashboard/bookmarks/EditorCard.tsx
+++ b/apps/web/components/dashboard/bookmarks/EditorCard.tsx
@@ -13,7 +13,7 @@ import {
   useBookmarkLayout,
   useBookmarkLayoutSwitch,
 } from "@/lib/userLocalSettings/bookmarksLayout";
-import { cn } from "@/lib/utils";
+import { cn, getOS } from "@/lib/utils";
 import { zodResolver } from "@hookform/resolvers/zod";
 import { useForm } from "react-hook-form";
 import { z } from "zod";
@@ -123,6 +123,7 @@ export default function EditorCard({ className }: { className?: string }) {
 
   const onSubmit: SubmitHandler<z.infer<typeof formSchema>> = (data) => {
     const text = data.text.trim();
+    if (!text.length) return;
     try {
       tryToImportUrls(text);
     } catch (e) {
@@ -162,12 +163,14 @@ export default function EditorCard({ className }: { className?: string }) {
     }
   };
 
+  const OS = getOS();
+
   return (
     <Form {...form}>
       <form
         className={cn(
           className,
-          "flex flex-col gap-2 rounded-xl bg-card p-4",
+          "relative flex flex-col gap-2 rounded-xl bg-card p-4",
           cardHeight,
         )}
         onSubmit={form.handleSubmit(onSubmit, onError)}
@@ -187,7 +190,7 @@ export default function EditorCard({ className }: { className?: string }) {
               ref={inputRef}
               disabled={isPending}
               className={cn(
-                "h-full w-full border-none text-lg focus-visible:ring-0",
+                "h-full w-full border-none p-0 text-lg focus-visible:ring-0",
                 { "resize-none": bookmarkLayout !== "list" },
               )}
               placeholder={
@@ -216,7 +219,7 @@ export default function EditorCard({ className }: { className?: string }) {
           {form.formState.dirtyFields.text
             ? demoMode
               ? "Submissions are disabled"
-              : "Press ⌘ + Enter to Save"
+              : `Save (${OS === "macos" ? "⌘" : "Ctrl"} + Enter)`
             : "Save"}
         </ActionButton>
 
diff --git a/apps/web/components/ui/info-tooltip.tsx b/apps/web/components/ui/info-tooltip.tsx
index eeace885..0254aa80 100644
--- a/apps/web/components/ui/info-tooltip.tsx
+++ b/apps/web/components/ui/info-tooltip.tsx
@@ -21,7 +21,11 @@ export default function InfoTooltip({
     <Tooltip>
       <TooltipTrigger asChild>
         {variant === "tip" ? (
-          <Info className={cn("cursor-pointer", className)} size={size} />
+          <Info
+            color="#494949"
+            className={cn("cursor-pointer", className)}
+            size={size}
+          />
         ) : (
           <HelpCircle className={cn("cursor-pointer", className)} size={size} />
         )}
diff --git a/apps/web/lib/utils.ts b/apps/web/lib/utils.ts
index 88283f01..12207765 100644
--- a/apps/web/lib/utils.ts
+++ b/apps/web/lib/utils.ts
@@ -5,3 +5,27 @@ import { twMerge } from "tailwind-merge";
 export function cn(...inputs: ClassValue[]) {
   return twMerge(clsx(inputs));
 }
+
+export type OS = "macos" | "ios" | "windows" | "android" | "linux" | null;
+
+export function getOS() {
+  if (typeof window === "undefined") return;
+  const userAgent = window.navigator.userAgent.toLowerCase();
+  const macosPlatforms = /(macintosh|macintel|macppc|mac68k|macos)/i;
+  const windowsPlatforms = /(win32|win64|windows|wince)/i;
+  const iosPlatforms = /(iphone|ipad|ipod)/i;
+  let os: OS = null;
+
+  if (macosPlatforms.test(userAgent)) {
+    os = "macos";
+  } else if (iosPlatforms.test(userAgent)) {
+    os = "ios";
+  } else if (windowsPlatforms.test(userAgent)) {
+    os = "windows";
+  } else if (/android/.test(userAgent)) {
+    os = "android";
+  } else if (!os && /linux/.test(userAgent)) {
+    os = "linux";
+  }
+  return os;
+}
-- 
cgit v1.2.3-70-g09d2