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

Commit

Permalink
Fixes #4738 - Remove subtitle link favicon (#4739)
Browse files Browse the repository at this point in the history
  • Loading branch information
punamdahiya authored and jaredhirsch committed Aug 3, 2018
1 parent 84b122f commit b8705d3
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions server/src/pages/shot/view.js
Original file line number Diff line number Diff line change
Expand Up @@ -400,12 +400,6 @@ class Body extends React.Component {
renderGetFirefox = renderExtensionNotification = false;
}

let favicon = null;
if (shot.favicon) {
// We use background-image so if the image is broken it just doesn't show:
favicon = <div style={{backgroundImage: `url("${shot.favicon}")`}} className="favicon" />;
}

const noText = this.props.abTests && this.props.abTests.downloadText
&& this.props.abTests.downloadText.value === "no-download-text";
return (
Expand All @@ -418,7 +412,6 @@ class Body extends React.Component {
<div className="shot-info">
<EditableTitle title={shot.title} isOwner={this.props.isOwner} />
<div className="shot-subtitle">
{ this.state.isChangingExpire ? null : favicon }
{ linkTextShort && !this.state.isChangingExpire ? <a className="subtitle-link" rel="noopener noreferrer" href={ shot.url } target="_blank" onClick={ this.onClickOrigUrl.bind(this, "navbar") }>{ linkTextShort }</a> : null }
{ this.state.isChangingExpire ? null : <span className="time-diff">{ timeDiff }</span> }
{ expiresDiff }
Expand Down

0 comments on commit b8705d3

Please sign in to comment.