Skip to content

Commit 8ddf164

Browse files
committed
refactor to fix codeclimate issue
1 parent 10fb5be commit 8ddf164

File tree

2 files changed

+25
-23
lines changed

2 files changed

+25
-23
lines changed

public/js/sidebar.js

Lines changed: 24 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Component } from 'react';
1+
import React, { Component } from 'react';
22
import _ from 'underscore';
33

44
import downloadFASTA from './download_fasta';
@@ -353,6 +353,7 @@ export default class extends Component {
353353
}
354354

355355
sharingPanelJSX() {
356+
const link_class = 'btn-link cursor-pointer';
356357
return (
357358
<div className="sharing-panel">
358359
<div className="section-header-sidebar">
@@ -361,29 +362,30 @@ export default class extends Component {
361362
</h4>
362363
</div>
363364
<ul className="nav">
364-
{!this.props.cloudSharingEnabled ?
365-
<>
366-
<li>
367-
<a id="copyURL" className="btn-link copy-URL cursor-pointer" data-toggle="tooltip"
368-
onClick={this.copyURL}>
369-
<i className="fa fa-copy"></i> Copy URL to clipboard
370-
</a>
371-
</li>
365+
{
366+
!this.props.cloudSharingEnabled ?
367+
<React.Fragment>
368+
<li>
369+
<a id="copyURL" className={`${link_class} copy-URL`} data-toggle="tooltip"
370+
onClick={this.copyURL}>
371+
<i className="fa fa-copy"></i> Copy URL to clipboard
372+
</a>
373+
</li>
374+
<li>
375+
<a id="sendEmail" className={`${link_class} email-URL`} data-toggle="tooltip"
376+
title="Send by email" href={asMailtoHref(this.props.data.querydb, this.props.data.program, this.props.data.queries, window.location.href)}
377+
target="_blank" rel="noopener noreferrer">
378+
<i className="fa fa-envelope"></i> Send by email
379+
</a>
380+
</li>
381+
</React.Fragment> :
372382
<li>
373-
<a id="sendEmail" className="btn-link email-URL cursor-pointer" data-toggle="tooltip"
374-
title="Send by email" href={asMailtoHref(this.props.data.querydb, this.props.data.program, this.props.data.queries, window.location.href)}
375-
target="_blank" rel="noopener noreferrer">
376-
<i className="fa fa-envelope"></i> Send by email
377-
</a>
378-
</li>
379-
</> :
380-
<li>
381-
<button className="btn-link cloud-Post cursor-pointer" data-toggle="tooltip"
382-
title="Upload results to SequenceServer Cloud where it will become accessable
383+
<button id="shareToCloud" className={`${link_class} cloud-Post`} data-toggle="tooltip"
384+
title="Upload results to SequenceServer Cloud where it will become accessable
383385
to everyone who has a link." onClick={this.shareCloudInit}>
384-
<i className="fa fa-cloud"></i> Share to cloud
385-
</button>
386-
</li>
386+
<i className="fa fa-cloud"></i> Share to cloud
387+
</button>
388+
</li>
387389
}
388390
</ul>
389391
{

0 commit comments

Comments
 (0)