Merge pull request #18888 from storybookjs/pocka/fix/18829-outdated-s…

…nippet-copied Fix copy button copying outdated snippet
storybookjs · Sep 13, 2022 · ae61864 · ae61864
2 parents cb19030 + 191103a
commit ae61864
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 15 deletions.
diff --git a/code/frameworks/vue3-vite/preset.js b/code/frameworks/vue3-vite/preset.js
@@ -1,2 +1,2 @@
-console.log('vue3-vite preset!')
+console.log('vue3-vite preset!');
 module.exports = require('./dist/preset');
diff --git a/code/lib/builder-manager/src/types.ts b/code/lib/builder-manager/src/types.ts
@@ -7,10 +7,9 @@ export interface Stats {
   toJson: () => any;
 }
 
-export type WithRequiredProperty<Type, Key extends keyof Type> = Type &
-  {
-    [Property in Key]-?: Type[Property];
-  };
+export type WithRequiredProperty<Type, Key extends keyof Type> = Type & {
+  [Property in Key]-?: Type[Property];
+};
 
 export type ManagerBuilder = Builder<
   WithRequiredProperty<BuildOptions, 'outdir'> & { entryPoints: string[] },

diff --git a/code/lib/builder-vite/src/codegen-entries.ts b/code/lib/builder-vite/src/codegen-entries.ts
@@ -6,7 +6,9 @@ import type { ExtendedOptions } from './types';
 import { listStories } from './list-stories';
 
 const absoluteFilesToImport = (files: string[], name: string) =>
-  files.map((el, i) => `import ${name ? `* as ${name}_${i} from ` : ''}'/@fs/${normalizePath(el)}'`).join('\n');
+  files
+    .map((el, i) => `import ${name ? `* as ${name}_${i} from ` : ''}'/@fs/${normalizePath(el)}'`)
+    .join('\n');
 
 export async function generateVirtualStoryEntryCode(options: ExtendedOptions) {
   const storyEntries = await listStories(options);

diff --git a/code/lib/components/src/syntaxhighlighter/syntaxhighlighter.tsx b/code/lib/components/src/syntaxhighlighter/syntaxhighlighter.tsx
@@ -218,15 +218,18 @@ export const SyntaxHighlighter: FC<SyntaxHighlighterProps> = ({
   const highlightableCode = formatter ? formatter(format, children) : children.trim();
   const [copied, setCopied] = useState(false);
 
-  const onClick = useCallback((e: MouseEvent<HTMLButtonElement>) => {
-    e.preventDefault();
-    copyToClipboard(highlightableCode)
-      .then(() => {
-        setCopied(true);
-        globalWindow.setTimeout(() => setCopied(false), 1500);
-      })
-      .catch(logger.error);
-  }, []);
+  const onClick = useCallback(
+    (e: MouseEvent<HTMLButtonElement>) => {
+      e.preventDefault();
+      copyToClipboard(highlightableCode)
+        .then(() => {
+          setCopied(true);
+          globalWindow.setTimeout(() => setCopied(false), 1500);
+        })
+        .catch(logger.error);
+    },
+    [highlightableCode]
+  );
   const renderer = wrapRenderer(rest.renderer, showLineNumbers);
 
   return (