Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Commit

Permalink
Fix #1659, rename #share-button to #toggle-share. Remove some styles …
Browse files Browse the repository at this point in the history
…that appeared to be for the share button, but didn't apply to anything
  • Loading branch information
ianb committed Oct 13, 2016
1 parent ae47ad9 commit 19863ea
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 15 deletions.
4 changes: 2 additions & 2 deletions server/src/views/frame.js
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down Expand Up @@ -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) }
Expand Down
13 changes: 0 additions & 13 deletions static/css/styles.scss
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 19863ea

Please sign in to comment.