Merge pull request #289 from devtron-labs/feat/kubeconfig-rb

fix: dont use internal trigger if controlled from outside

Author: Elessar1802

package-lock.json

@@ -1,6 +1,6 @@
 {
     "name": "@devtron-labs/devtron-fe-common-lib",
-    "version": "0.2.17-patch-2",
+    "version": "0.2.17-beta-4",
     "description": "Supporting common component library",
     "type": "module",
     "main": "dist/index.js",

package.json

@@ -34,14 +34,16 @@ export default function ClipboardButton({
     content,
     copiedTippyText = 'Copied!',
     duration = 1000,
-    trigger = false,
+    trigger,
     setTrigger = noop,
     rootClassName = '',
     iconSize = 16,
 }: ClipboardProps) {
     const [copied, setCopied] = useState<boolean>(false)
     const [enableTippy, setEnableTippy] = useState<boolean>(false)
 
+    const isTriggerUndefined = typeof trigger === 'undefined'
+
     const handleTextCopied = () => {setCopied(true)}
     const handleEnableTippy = () => setEnableTippy(true)
     const handleDisableTippy = () => setEnableTippy(false)
@@ -66,7 +68,7 @@ export default function ClipboardButton({
     }, [copied, duration, setTrigger])
 
     useEffect(() => {
-        if (trigger) {
+        if (!isTriggerUndefined && trigger) {
             setCopied(true)
             handleCopyContent()
         }
@@ -84,7 +86,7 @@ export default function ClipboardButton({
                 className={`dc__outline-none-imp p-0 flex dc__transparent--unstyled dc__no-border ${rootClassName}`}
                 onMouseEnter={handleEnableTippy}
                 onMouseLeave={handleDisableTippy}
-                onClick={handleCopyContent}
+                onClick={isTriggerUndefined && handleCopyContent}
             >
                 {copied ? <Check className={iconClassName} /> : <ICCopy className={iconClassName} />}
             </button>