From 61ced65b756d9120e77155a849877580e4cbcc64 Mon Sep 17 00:00:00 2001
From: André Oliveira <37463445+SpecialAro@users.noreply.github.com>
Date: Sat, 17 Jun 2023 15:37:45 +0100
Subject: fix: changelog links opening in-app instead of browser (#1241)

---
 .../releaseNotes/ReleaseNotesDashboard.tsx         | 31 +++++++++-------------
 1 file changed, 12 insertions(+), 19 deletions(-)

(limited to 'src')

diff --git a/src/components/settings/releaseNotes/ReleaseNotesDashboard.tsx b/src/components/settings/releaseNotes/ReleaseNotesDashboard.tsx
index d0be82312..ff7c45bb1 100644
--- a/src/components/settings/releaseNotes/ReleaseNotesDashboard.tsx
+++ b/src/components/settings/releaseNotes/ReleaseNotesDashboard.tsx
@@ -2,7 +2,6 @@ import { Component } from 'react';
 import { observer } from 'mobx-react';
 import { defineMessages, injectIntl } from 'react-intl';
 import Markdown from 'markdown-to-jsx';
-import { openExternalUrl } from '../../../helpers/url-helpers';
 import { ferdiumVersion } from '../../../environment-remote';
 import {
   getFerdiumVersion,
@@ -53,25 +52,12 @@ class ReleaseNotesDashboard extends Component<IProps> {
     this.setState({
       data,
     });
-
-    for (const link of document.querySelectorAll('.releasenotes__body a')) {
-      link.addEventListener('click', this.handleClick.bind(this), false);
-    }
-  }
-
-  handleClick(e) {
-    e.preventDefault();
-    openExternalUrl(e.target.href);
   }
 
-  componentWillUnmount() {
-    document.removeEventListener(
-      'click',
-      // eslint-disable-next-line unicorn/no-invalid-remove-event-listener
-      this.handleClick.bind(this),
-      false,
-    );
-  }
+  overrideAnchor = props => (
+    // eslint-disable-next-line jsx-a11y/anchor-has-content
+    <a {...props} target="_blank" rel="noopener noreferrer" />
+  );
 
   render() {
     const { intl } = this.props;
@@ -89,7 +75,14 @@ class ReleaseNotesDashboard extends Component<IProps> {
           </span>
         </div>
         <div className="settings__body releasenotes__body">
-          <Markdown options={{ wrapper: 'article' }}>{data}</Markdown>
+          <Markdown
+            options={{
+              wrapper: 'article',
+              overrides: { a: this.overrideAnchor },
+            }}
+          >
+            {data}
+          </Markdown>
         </div>
       </div>
     );
-- 
cgit v1.2.3-54-g00ecf