Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

Add noopener noreferrer attribute to links

Browse Source
This commit is contained in:
Pavel Dohnal
2019-01-23 13:16:53 +01:00
parent f054f7ba1f
commit 4646086660
10 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.eslintrc.es6.json
View File

@ -32,8 +32,7 @@
"react/default-props-match-prop-types": 0, // todo
"react/no-access-state-in-setstate": 0, // todo
"react/jsx-closing-tag-location": 0, // todo
"react/require-default-props": 0, // todo
"react/jsx-no-target-blank": 0 // todo
"react/require-default-props": 0 // todo
}
}

2
assets/js/src/help/system_status.jsx
View File

@ -36,7 +36,7 @@ function renderCronSection(data) {
<div>
<h2>{MailPoet.I18n.t('systemStatusCronTitle')}</h2>
<p>
<a href={url} target="_blank">{url}</a>
<a href={url} target="_blank" rel="noopener noreferrer">{url}</a>
</p>
{renderStatusMessage(status, MailPoet.I18n.t('systemStatusCronConnectionUnsuccessfulInfo'), '//beta.docs.mailpoet.com/article/231-sending-does-not-work')}
</div>

1
assets/js/src/help/tasks_list/tasks_list_data_row.jsx
View File

@ -17,6 +17,7 @@ const TasksListDataRow = props => (
data-newsletter-id={props.task.newsletter.newsletter_id}
data-queue-id={props.task.newsletter.queue_id}
target="_blank"
rel="noopener noreferrer"
>
{props.task.newsletter.subject || MailPoet.I18n.t('preview')}
</a>

1
assets/js/src/newsletters/listings/mixins.jsx
View File

@ -310,6 +310,7 @@ const StatisticsMixin = {
<a
href={improveStatsKBLink}
target="_blank"
rel="noopener noreferrer"
className="mailpoet_stat_link_small"
>
{MailPoet.I18n.t('improveThisLinkText')}

2
assets/js/src/newsletters/listings/notification.jsx
View File

@ -111,7 +111,7 @@ const newsletterActions = [
name: 'view',
link: function link(newsletter) {
return (
<a href={newsletter.preview_url} target="_blank">
<a href={newsletter.preview_url} target="_blank" rel="noopener noreferrer">
{MailPoet.I18n.t('preview')}
</a>
);

3
assets/js/src/newsletters/listings/notification_history.jsx
View File

@ -49,7 +49,7 @@ let newsletterActions = [
name: 'view',
link: function link(newsletter) {
return (
<a href={newsletter.preview_url} target="_blank">
<a href={newsletter.preview_url} target="_blank" rel="noopener noreferrer">
{MailPoet.I18n.t('preview')}
</a>
);
@ -88,6 +88,7 @@ const NewsletterListNotificationHistory = createReactClass({ // eslint-disable-l
<a
href={newsletter.preview_url}
target="_blank"
rel="noopener noreferrer"
>
{ newsletter.queue.newsletter_rendered_subject || newsletter.subject }
</a>

2
assets/js/src/newsletters/listings/standard.jsx
View File

@ -125,7 +125,7 @@ let newsletterActions = [
name: 'view',
link: function link(newsletter) {
return (
<a href={newsletter.preview_url} target="_blank">
<a href={newsletter.preview_url} target="_blank" rel="noopener noreferrer">
{MailPoet.I18n.t('preview')}
</a>
);

2
assets/js/src/newsletters/listings/welcome.jsx
View File

@ -110,7 +110,7 @@ let newsletterActions = [
name: 'view',
link: function link(newsletter) {
return (
<a href={newsletter.preview_url} target="_blank">
<a href={newsletter.preview_url} target="_blank" rel="noopener noreferrer">
{MailPoet.I18n.t('preview')}
</a>
);

2
assets/js/src/newsletters/types.jsx
View File

@ -189,7 +189,7 @@ class NewsletterTypes extends React.Component {
</h3>
<p>{type.description}</p>
{ type.videoGuide && (
<a className={badgeClassName} href={type.videoGuide} target="_blank">
<a className={badgeClassName} href={type.videoGuide} target="_blank" rel="noopener noreferrer">
<span className="dashicons dashicons-format-video" />
{MailPoet.I18n.t('seeVideoGuide')}
</a>

1
assets/js/src/newsletters/types/automatic_emails/event.jsx
View File

@ -23,6 +23,7 @@ class AutomaticEmailEvent extends React.PureComponent {
<a
href={event.actionButtonLink}
target="_blank"
rel="noopener noreferrer"
>
{event.actionButtonTitle}
</a>