Merge pull request #277 from kayleemann/master

Support case where showPreview is false but showSkinTones is true. Also add prepare script.
missive · Mar 8, 2019 · a871cac · a871cac
2 parents ce77721 + 22aa42a
commit a871cac
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/package.json b/package.json
@@ -76,7 +76,8 @@
     "prepublishOnly": "npm run build",
     "storybook": "start-storybook -p 6006",
     "build-storybook": "build-storybook",
-    "prettier": "prettier --write \"{src,scripts,spec}/**/*.js\""
+    "prettier": "prettier --write \"{src,scripts,spec}/**/*.js\"",
+    "prepare": "npm run build:dist"
   },
   "size-limit": [
     {

diff --git a/src/components/picker/nimble-picker.js b/src/components/picker/nimble-picker.js
@@ -566,14 +566,15 @@ export default class NimblePicker extends React.PureComponent {
           })}
         </div>
 
-        {showPreview && (
+        {(showPreview || showSkinTones) && (
           <div className="emoji-mart-bar">
             <Preview
               ref={this.setPreviewRef}
               data={this.data}
               title={title}
               emoji={emoji}
               showSkinTones={showSkinTones}
+              showPreview={showPreview}
               emojiProps={{
                 native: native,
                 size: 38,

diff --git a/src/components/preview.js b/src/components/preview.js
@@ -23,9 +23,10 @@ export default class Preview extends React.PureComponent {
         title,
         emoji: idleEmoji,
         i18n,
+        showPreview,
       } = this.props
 
-    if (emoji) {
+    if (emoji && showPreview) {
       var emojiData = getData(emoji, null, null, this.data),
         { emoticons = [] } = emojiData,
         knownEmoticons = [],