From 19863ea5da1c6206f7ec93e78fd0c9b5b9645a24 Mon Sep 17 00:00:00 2001
From: Ian Bicking <ianb@colorstudy.com>
Date: Thu, 13 Oct 2016 15:47:40 -0500
Subject: [PATCH] Fix #1659, rename #share-button to #toggle-share.  Remove
 some styles that appeared to be for the share button, but didn't apply to
 anything

---
 server/src/views/frame.js |  4 ++--
 static/css/styles.scss    | 13 -------------
 2 files changed, 2 insertions(+), 15 deletions(-)

diff --git a/server/src/views/frame.js b/server/src/views/frame.js
index 2ebd8b8f43..39cf4ca4f3 100644
--- a/server/src/views/frame.js
+++ b/server/src/views/frame.js
@@ -378,7 +378,7 @@ class Frame extends React.Component {
   unsharePanelHandler(e) {
     let el = e.target;
     while (el) {
-      if (el.id === "share-buttons-panel" || el.id === "share-button") {
+      if (el.id === "share-buttons-panel" || el.id === "toggle-share") {
         // A click in the share panel itself
         return;
       }
@@ -573,7 +573,7 @@ class Frame extends React.Component {
             </div>
           </div>
           <div className="more-shot-actions right">
-            <button className="button primary" id="share-button" onClick={ this.onClickShareButton.bind(this) }>
+            <button className="button primary" id="toggle-share" onClick={ this.onClickShareButton.bind(this) }>
               Share
             </button>
             <a className="button secondary" href={ clipUrl } onClick={ this.onClickDownload.bind(this) }
diff --git a/static/css/styles.scss b/static/css/styles.scss
index b93fe52062..8ad6e58377 100644
--- a/static/css/styles.scss
+++ b/static/css/styles.scss
@@ -152,19 +152,6 @@ a:hover {
   font-size: 12px;
 }
 
-.share-button {
-  margin-left: 15px;
-  background: #0996f8;
-  color: #fff;
-  border-radius: 4px;
-  border: 1px solid #0675d3;
-  height: 32px;
-  cursor: pointer;
-  vertical-align: middle;
-  padding: 6px 15px;
-  font-size: 12px;
-}
-
 .trash-button {
   margin-left: 15px;
   border-radius: 4px;