feature: Made copying manifest url to clipboard better

Composer/packages/client/src/pages/publish/BotStatusList.tsx

@@ -16,14 +16,57 @@ import { ActionButton, IconButton } from 'office-ui-fabric-react/lib/Button';
 import { SharedColors } from '@uifabric/fluent-theme';
 import { FontSizes } from '@uifabric/styling';
 import get from 'lodash/get';
-import { ITextField, TextField } from 'office-ui-fabric-react/lib/TextField';
+import { useCopyToClipboard } from '@bfc/ui-shared';
+import { Stack } from 'office-ui-fabric-react/lib/Stack';
+import { Callout } from 'office-ui-fabric-react';
 
 import { ApiStatus } from '../../utils/publishStatusPollingUpdater';
 
 import { PublishStatusList } from './PublishStatusList';
 import { detailList, listRoot, tableView } from './styles';
 import { BotPublishHistory, BotStatus } from './type';
 
+const copiedCalloutStyles = {
+  root: {
+    padding: '10px',
+  },
+};
+
+type SkillManifestUrlFieldProps = {
+  url: string;
+};
+
+const SkillManifestUrlField = ({ url }: SkillManifestUrlFieldProps) => {
+  const { isCopiedToClipboard, copyTextToClipboard, resetIsCopiedToClipboard } = useCopyToClipboard(url);
+
+  const calloutTarget = useRef<EventTarget>();
+  return (
+    <Stack>
+      <ActionButton
+        className="skill-manifest-copy-button"
+        title={url}
+        onClick={(e) => {
+          calloutTarget.current = e.target;
+          copyTextToClipboard?.();
+        }}
+      >
+        {formatMessage('Copy Skill Manifest URL')}
+      </ActionButton>
+      {isCopiedToClipboard && (
+        <Callout
+          setInitialFocus
+          calloutMaxWidth={200}
+          styles={copiedCalloutStyles}
+          target={calloutTarget}
+          onDismiss={resetIsCopiedToClipboard}
+        >
+          {formatMessage('Skill manifest URL was copied to the clipboard')}
+        </Callout>
+      )}
+    </Stack>
+  );
+};
+
 export type BotStatusListProps = {
   botStatusList: BotStatus[];
   botPublishHistoryList: BotPublishHistory;
@@ -49,25 +92,6 @@ export const BotStatusList: React.FC<BotStatusListProps> = ({
 }) => {
   const [expandedBotIds, setExpandedBotIds] = useState<Record<string, boolean>>({});
   const [currentSort, setSort] = useState({ key: 'Bot', descending: true });
-  const [clipboardText, setClipboardText] = useState('');
-  const clipboardTextFieldRef = useRef<ITextField>(null);
-
-  const copyStringToClipboard = (value?: string) => {
-    try {
-      if (clipboardTextFieldRef.current) {
-        setClipboardText(value || '');
-        setTimeout(() => {
-          if (clipboardTextFieldRef.current) {
-            clipboardTextFieldRef.current.select();
-            document.execCommand('copy');
-          }
-        }, 10);
-      }
-    } catch (e) {
-      // eslint-disable-next-line no-console
-      console.error('Something went wrong when copying to the clipboard.', e, location);
-    }
-  };
 
   const displayedItems: BotStatus[] = useMemo(() => {
     if (currentSort.key !== 'Bot') return botStatusList;
@@ -273,18 +297,7 @@ export const BotStatusList: React.FC<BotStatusListProps> = ({
       maxWidth: 134,
       data: 'string',
       onRender: (item: BotStatus) => {
-        return (
-          item?.skillManifestUrl && (
-            <ActionButton
-              title={item.skillManifestUrl}
-              onClick={() => {
-                copyStringToClipboard(item.skillManifestUrl);
-              }}
-            >
-              {formatMessage('Copy Skill Manifest URL')}
-            </ActionButton>
-          )
-        );
+        return item?.skillManifestUrl && <SkillManifestUrlField url={item.skillManifestUrl} />;
       },
       isPadded: true,
     },
@@ -365,12 +378,6 @@ export const BotStatusList: React.FC<BotStatusListProps> = ({
           onRenderRow={renderTableRow}
         />
       </div>
-      <TextField
-        readOnly
-        componentRef={clipboardTextFieldRef}
-        styles={{ root: { display: 'none' } }}
-        value={clipboardText}
-      />
     </div>
   );
 };

Composer/packages/lib/ui-shared/src/hooks/index.ts

@@ -0,0 +1,4 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+export * from './useCopyToClipboard';

Composer/packages/lib/ui-shared/src/hooks/useCopyToClipboard.ts

@@ -0,0 +1,55 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+import * as React from 'react';
+
+/**
+ * Creates a textarea at then end of the document body then
+ * uses that to copy the specified text to the cliboard.
+ */
+
+export const copyToClipboard = (text: string) => {
+  // Remember the current selection
+  const rangeCount = document?.getSelection()?.rangeCount || 0;
+  const selected = rangeCount > 0 ? document?.getSelection()?.getRangeAt(0) : false;
+
+  // Create an offscreen textarea and copy the text
+  const element = document.createElement('textarea');
+  element.value = text;
+  element.setAttribute('readonly', '');
+  element.style.position = 'absolute';
+  element.style.left = '-9999px';
+  document.body.appendChild(element);
+  element.select();
+  const success = document.execCommand('copy');
+  document.body.removeChild(element);
+
+  // Restore the previous selection
+  if (document?.getSelection() && selected) {
+    document?.getSelection()?.removeAllRanges();
+    document?.getSelection()?.addRange(selected);
+  }
+
+  if (!success) {
+    throw new Error('There was a problem copying to the clipboard.');
+  }
+};
+
+/**
+ * Hook to copy text to the clipboard.
+ */
+export const useCopyToClipboard = (text: string) => {
+  const [isCopiedToClipboard, setIsCopiedToClipboard] = React.useState(false);
+
+  const copyTextToClipboard = React.useCallback(() => {
+    copyToClipboard(text);
+    setIsCopiedToClipboard(true);
+  }, [text]);
+
+  const resetIsCopiedToClipboard = () => setIsCopiedToClipboard(false);
+
+  // When the text changes, reset copied.
+  React.useEffect(() => () => resetIsCopiedToClipboard(), [text]);
+
+  return { isCopiedToClipboard, copyTextToClipboard, resetIsCopiedToClipboard };
+};

Composer/packages/lib/ui-shared/src/index.tsx

@@ -2,5 +2,6 @@
 // Licensed under the MIT License.
 
 export * from './components';
+export * from './hooks';
 export * from './styled';
 export * from './constants';

Composer/packages/server/src/locales/en-US.json

@@ -3299,6 +3299,9 @@
   "skill_manifest_endpoint_is_configured_improperly_e083731d": {
     "message": "Skill manifest endpoint is configured improperly"
   },
+  "skill_manifest_url_was_copied_to_the_clipboard_4cfad630": {
+    "message": "Skill manifest URL was copied to the clipboard"
+  },
   "skillname_manifest_ef3d9fed": {
     "message": "{ skillName } Manifest"
   },