From 4f4d0e0fc29c156393c42fd179495db301a928d0 Mon Sep 17 00:00:00 2001
From: Pascal Mugnier <pascal.mugnier@sonarsource.com>
Date: Fri, 27 Jul 2018 11:57:40 +0200
Subject: [PATCH] SONAR-11076 add more docs links and tool tips

---
 .../quality-gates/components/Conditions.tsx   | 10 ++-
 .../apps/quality-profiles/home/PageHeader.tsx |  7 ++
 .../apps/securityReports/components/App.tsx   |  7 ++
 .../__tests__/__snapshots__/App-test.tsx.snap | 65 +++++++++++++++++++
 4 files changed, 88 insertions(+), 1 deletion(-)

diff --git a/server/sonar-web/src/main/js/apps/quality-gates/components/Conditions.tsx b/server/sonar-web/src/main/js/apps/quality-gates/components/Conditions.tsx
index 88733a585e9..0f1c8cc3a49 100644
--- a/server/sonar-web/src/main/js/apps/quality-gates/components/Conditions.tsx
+++ b/server/sonar-web/src/main/js/apps/quality-gates/components/Conditions.tsx
@@ -123,7 +123,15 @@ export default class Conditions extends React.PureComponent<Props> {
           <table className="data zebra zebra-hover" id="quality-gate-conditions">
             <thead>
               <tr>
-                <th className="nowrap">{translate('quality_gates.conditions.metric')}</th>
+                <th className="nowrap">
+                  <div className="display-inline-flex-center">
+                    {translate('quality_gates.conditions.metric')}
+                    <DocTooltip
+                      className="spacer-left"
+                      doc={import(/* webpackMode: "eager" */ 'Docs/tooltips/quality-gates/metric.md')}
+                    />
+                  </div>
+                </th>
                 <th className="thin nowrap">{translate('quality_gates.conditions.leak')}</th>
                 <th className="thin nowrap">{translate('quality_gates.conditions.operator')}</th>
                 <th className="thin nowrap">{translate('quality_gates.conditions.warning')}</th>
diff --git a/server/sonar-web/src/main/js/apps/quality-profiles/home/PageHeader.tsx b/server/sonar-web/src/main/js/apps/quality-profiles/home/PageHeader.tsx
index 1373fa0f309..4bfb1c02c77 100644
--- a/server/sonar-web/src/main/js/apps/quality-profiles/home/PageHeader.tsx
+++ b/server/sonar-web/src/main/js/apps/quality-profiles/home/PageHeader.tsx
@@ -19,6 +19,7 @@
  */
 import * as React from 'react';
 import * as PropTypes from 'prop-types';
+import { Link } from 'react-router';
 import CreateProfileForm from './CreateProfileForm';
 import RestoreProfileForm from './RestoreProfileForm';
 import { Profile } from '../types';
@@ -100,6 +101,12 @@ export default class PageHeader extends React.PureComponent<Props, State> {
           {translate('quality_profiles.intro1')}
           <br />
           {translate('quality_profiles.intro2')}
+          <Link
+            className="spacer-left"
+            target="_blank"
+            to={{ pathname: '/documentation/quality-profiles' }}>
+            {translate('learn_more')}
+          </Link>
         </div>
 
         {this.state.restoreFormOpen && (
diff --git a/server/sonar-web/src/main/js/apps/securityReports/components/App.tsx b/server/sonar-web/src/main/js/apps/securityReports/components/App.tsx
index 2395da7da87..653f0ae26ff 100755
--- a/server/sonar-web/src/main/js/apps/securityReports/components/App.tsx
+++ b/server/sonar-web/src/main/js/apps/securityReports/components/App.tsx
@@ -20,6 +20,7 @@
 import * as React from 'react';
 import * as PropTypes from 'prop-types';
 import Helmet from 'react-helmet';
+import { Link } from 'react-router';
 import VulnerabilityList from './VulnerabilityList';
 import Suggestions from '../../../app/components/embed-docs-modal/Suggestions';
 import { translate } from '../../../helpers/l10n';
@@ -135,6 +136,12 @@ export default class App extends React.PureComponent<Props, State> {
           <h1 className="page-title">{translate('security_reports', type, 'page')}</h1>
           <div className="page-description">
             {translate('security_reports', type, 'description')}
+            <Link
+              className="spacer-left"
+              target="_blank"
+              to={{ pathname: '/documentation/security-reports' }}>
+              {translate('learn_more')}
+            </Link>
           </div>
         </header>
         <div className="display-inline-flex-center">
diff --git a/server/sonar-web/src/main/js/apps/securityReports/components/__tests__/__snapshots__/App-test.tsx.snap b/server/sonar-web/src/main/js/apps/securityReports/components/__tests__/__snapshots__/App-test.tsx.snap
index b0059c48b81..7a82485d1b7 100644
--- a/server/sonar-web/src/main/js/apps/securityReports/components/__tests__/__snapshots__/App-test.tsx.snap
+++ b/server/sonar-web/src/main/js/apps/securityReports/components/__tests__/__snapshots__/App-test.tsx.snap
@@ -25,6 +25,19 @@ exports[`handle checkbox for cwe display 1`] = `
       className="page-description"
     >
       security_reports.owaspTop10.description
+      <Link
+        className="spacer-left"
+        onlyActiveOnIndex={false}
+        style={Object {}}
+        target="_blank"
+        to={
+          Object {
+            "pathname": "/documentation/security-reports",
+          }
+        }
+      >
+        learn_more
+      </Link>
     </div>
   </header>
   <div
@@ -95,6 +108,19 @@ exports[`handle checkbox for cwe display 2`] = `
       className="page-description"
     >
       security_reports.owaspTop10.description
+      <Link
+        className="spacer-left"
+        onlyActiveOnIndex={false}
+        style={Object {}}
+        target="_blank"
+        to={
+          Object {
+            "pathname": "/documentation/security-reports",
+          }
+        }
+      >
+        learn_more
+      </Link>
     </div>
   </header>
   <div
@@ -208,6 +234,19 @@ exports[`renders owaspTop10 1`] = `
       className="page-description"
     >
       security_reports.owaspTop10.description
+      <Link
+        className="spacer-left"
+        onlyActiveOnIndex={false}
+        style={Object {}}
+        target="_blank"
+        to={
+          Object {
+            "pathname": "/documentation/security-reports",
+          }
+        }
+      >
+        learn_more
+      </Link>
     </div>
   </header>
   <div
@@ -278,6 +317,19 @@ exports[`renders sansTop25 1`] = `
       className="page-description"
     >
       security_reports.sansTop25.description
+      <Link
+        className="spacer-left"
+        onlyActiveOnIndex={false}
+        style={Object {}}
+        target="_blank"
+        to={
+          Object {
+            "pathname": "/documentation/security-reports",
+          }
+        }
+      >
+        learn_more
+      </Link>
     </div>
   </header>
   <div
@@ -348,6 +400,19 @@ exports[`renders with cwe 1`] = `
       className="page-description"
     >
       security_reports.owaspTop10.description
+      <Link
+        className="spacer-left"
+        onlyActiveOnIndex={false}
+        style={Object {}}
+        target="_blank"
+        to={
+          Object {
+            "pathname": "/documentation/security-reports",
+          }
+        }
+      >
+        learn_more
+      </Link>
     </div>
   </header>
   <div
-- 
2.39.5