address review

ampproject · Sep 8, 2021 · 80fdc07 · 80fdc07
1 parent 77db078
commit 80fdc07
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 15 deletions.
diff --git a/extensions/amp-animation/0.1/storybook/Basic.amp.js b/extensions/amp-animation/0.1/storybook/Basic.amp.js
@@ -28,7 +28,7 @@ const KEYFRAMES_OPTIONS = {
   },
 };
 
-const keyframesOptionValues = Object.keys(KEYFRAMES_OPTIONS);
+const keyframesOptions = Object.keys(KEYFRAMES_OPTIONS);
 
 const BLOCK_STYLE = {
   background: 'blue',
@@ -45,11 +45,11 @@ export default {
   argTypes: {
     keyframesName: {
       control: {type: 'select'},
-      options: keyframesOptionValues,
+      options: keyframesOptions,
     },
   },
   args: {
-    keyframesName: keyframesOptionValues[0],
+    keyframesName: keyframesOptions[0],
     easing: 'cubic-bezier(0,0,.21,1)',
   },
 };

diff --git a/extensions/amp-brightcove/1.0/storybook/Basic.amp.js b/extensions/amp-brightcove/1.0/storybook/Basic.amp.js
@@ -14,28 +14,19 @@ export default {
     'data-video-id': 'ref:amp-docs-sample',
     'data-player-id': 'SyIOV8yWM',
     'data-account': '1290862519001',
+    'data-referrer': 'EXTERNAL_REFERRER',
     height: '270',
     width: '480',
   },
 };
 
 export const Default = (args) => {
-  return (
-    <amp-brightcove
-      data-referrer="EXTERNAL_REFERRER"
-      layout="responsive"
-      {...args}
-    ></amp-brightcove>
-  );
+  return <amp-brightcove layout="responsive" {...args}></amp-brightcove>;
 };
 
 export const WithPlaceholderAndFallback = (args) => {
   return (
-    <amp-brightcove
-      data-referrer="EXTERNAL_REFERRER"
-      layout="responsive"
-      {...args}
-    >
+    <amp-brightcove layout="responsive" {...args}>
       <div placeholder style="background:red">
         Placeholder. Loading content...
       </div>