Merge pull request #4049 from weseek/imprv/GW-6866-changing-the-cursor-when-hovering-over-a-link

imprv/gw6866 changing the cursor when hovering over a link

src/client/js/components/Admin/SlackIntegration/CustomBotWithProxySettings.jsx

@@ -66,11 +66,12 @@ const CustomBotWithProxySettings = (props) => {
   return (
     <>
       <h2 className="admin-setting-header mb-2">{t('admin:slack_integration.custom_bot_with_proxy_integration')}
-        <i
-          className="fa fa-external-link btn-link ml-2"
-          aria-hidden="true"
-          onClick={() => window.open(`${t('admin:slack_integration.docs_url.custom_bot_with_proxy')}`, '_blank')}
-        />
+        <a href={t('admin:slack_integration.docs_url.custom_bot_with_proxy')} target="_blank" rel="noopener noreferrer">
+          <i
+            className="fa fa-external-link btn-link ml-2"
+            aria-hidden="true"
+          />
+        </a>
       </h2>
 
       {slackAppIntegrations.length !== 0 && (

src/client/js/components/Admin/SlackIntegration/CustomBotWithoutProxySettings.jsx

@@ -21,11 +21,12 @@ const CustomBotWithoutProxySettings = (props) => {
   return (
     <>
       <h2 className="admin-setting-header">{t('admin:slack_integration.custom_bot_without_proxy_integration')}
-        <i
-          className="fa fa-external-link btn-link ml-2"
-          aria-hidden="true"
-          onClick={() => window.open(`${t('admin:slack_integration.docs_url.custom_bot_without_proxy')}`, '_blank')}
-        />
+        <a href={t('admin:slack_integration.docs_url.custom_bot_without_proxy')} target="_blank" rel="noopener noreferrer">
+          <i
+            className="fa fa-external-link btn-link ml-2"
+            aria-hidden="true"
+          />
+        </a>
       </h2>
 
       <CustomBotWithoutProxyConnectionStatus

src/client/js/components/Admin/SlackIntegration/OfficialBotSettings.jsx

@@ -47,11 +47,13 @@ const OfficialBotSettings = (props) => {
   return (
     <>
       <h2 className="admin-setting-header">{t('admin:slack_integration.official_bot_integration')}
-        <i
-          className="fa fa-external-link btn-link ml-2"
-          aria-hidden="true"
-          onClick={() => window.open(`${t('admin:slack_integration.docs_url.official_bot')}`, '_blank')}
-        />
+        <a href={t('admin:slack_integration.docs_url.official_bot')} target="_blank" rel="noopener noreferrer">
+          <i
+            className="fa fa-external-link btn-link ml-2"
+            aria-hidden="true"
+            onClick={() => window.open(`${t('admin:slack_integration.docs_url.official_bot')}`, '_blank')}
+          />
+        </a>
       </h2>
 
       {slackAppIntegrations.length !== 0 && (