From 8505229120ff34e19ef210e4c881b7e37a858a3b Mon Sep 17 00:00:00 2001
From: Glen Hunter <ghunter@sonatype.com>
Date: Thu, 6 Jan 2022 12:51:59 -0500
Subject: [PATCH 1/3] Show guidelines in NxAccordion RSC-843

---
 .../NxAccordion/NxAccordionPage.tsx           | 41 ++++++++++++++++++-
 1 file changed, 40 insertions(+), 1 deletion(-)

diff --git a/gallery/src/components/NxAccordion/NxAccordionPage.tsx b/gallery/src/components/NxAccordion/NxAccordionPage.tsx
index 65bf392453..562179c49d 100644
--- a/gallery/src/components/NxAccordion/NxAccordionPage.tsx
+++ b/gallery/src/components/NxAccordion/NxAccordionPage.tsx
@@ -5,7 +5,17 @@
  * distribution and is available at https://www.eclipse.org/legal/epl-2.0/.
  */
 import React from 'react';
-import { NxTable, NxInfoAlert, NxCode, NxTextLink, NxP, NxH3, NxTile } from '@sonatype/react-shared-components';
+import {
+  NxTable,
+  NxInfoAlert,
+  NxCode,
+  NxTextLink,
+  NxP,
+  NxH3,
+  NxTile,
+  NxH2,
+  NxList
+} from '@sonatype/react-shared-components';
 
 import { GalleryDescriptionTile, GalleryExampleTile } from '../../gallery-components/GalleryTiles';
 
@@ -32,6 +42,35 @@ const NxAccordionPage = () =>
         Note that this component is stateless – its open state must be tracked externally.
         See <NxCode>NxStatefulAccordion</NxCode> for a version which tracks its own open state.
       </NxP>
+      <NxH2>Guidelines</NxH2>
+        <NxH3>When to Use</NxH3>
+        <NxList bulleted>
+          <NxList.Item>Displaying and grouping additional information.</NxList.Item>
+          <NxList.Item>Adding granular control over the information on a given page.</NxList.Item>
+          <NxList.Item>Shortening pages to reduce scrolling.</NxList.Item>
+        </NxList>
+        <NxH3>When Not to Use</NxH3>
+        <NxList bulleted>
+          <NxList.Item>
+            Displaying critical system information or a primary action to be taken on the page. (for example,
+            {' '}<a href="/components/alert">alerts</a>, confirmation or cancellation buttons).
+          </NxList.Item>
+          <NxList.Item>
+            Displaying navigation elements such as <a href="/components/tabs">tabs</a>.
+          </NxList.Item>
+          <NxList.Item>
+            Displaying links pointing to sections of the same page, instead use a
+            {' '}<a href="/components/list">list</a>.
+          </NxList.Item>
+          <NxList.Item>
+            Creating hierarchy levels by nesting them within each other. If you need to add hierarchy to the
+            content use a <a href="/components/tree">tree</a>.
+          </NxList.Item>
+          <NxList.Item>
+            Displaying a set of visual components following the same style, prefer using Collapsable.(TODO: Add
+            link to collapsable).
+          </NxList.Item>
+        </NxList>
       <NxTile.Subsection>
         <NxTile.SubsectionHeader>
           <NxH3>NxAccordion</NxH3>

From abc48124caa1ceb5bcefb7388edf2f1e2bb35cfd Mon Sep 17 00:00:00 2001
From: Glen Hunter <ghunter@sonatype.com>
Date: Fri, 7 Jan 2022 10:38:14 -0500
Subject: [PATCH 2/3] Proper subsection for guidelines RSC-843

---
 gallery/src/components/NxAccordion/NxAccordionPage.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/gallery/src/components/NxAccordion/NxAccordionPage.tsx b/gallery/src/components/NxAccordion/NxAccordionPage.tsx
index 562179c49d..f4bff70911 100644
--- a/gallery/src/components/NxAccordion/NxAccordionPage.tsx
+++ b/gallery/src/components/NxAccordion/NxAccordionPage.tsx
@@ -42,7 +42,10 @@ const NxAccordionPage = () =>
         Note that this component is stateless – its open state must be tracked externally.
         See <NxCode>NxStatefulAccordion</NxCode> for a version which tracks its own open state.
       </NxP>
-      <NxH2>Guidelines</NxH2>
+      <NxTile.Subsection>
+        <NxTile.SubsectionHeader>
+          <NxH3>Guidelines</NxH3>
+        </NxTile.SubsectionHeader>
         <NxH3>When to Use</NxH3>
         <NxList bulleted>
           <NxList.Item>Displaying and grouping additional information.</NxList.Item>
@@ -71,6 +74,7 @@ const NxAccordionPage = () =>
             link to collapsable).
           </NxList.Item>
         </NxList>
+      </NxTile.Subsection>
       <NxTile.Subsection>
         <NxTile.SubsectionHeader>
           <NxH3>NxAccordion</NxH3>

From d203f326829ba33c839209e7af2bbc22ec5fc290 Mon Sep 17 00:00:00 2001
From: Glen Hunter <ghunter@sonatype.com>
Date: Fri, 7 Jan 2022 10:55:27 -0500
Subject: [PATCH 3/3] Use NxTextLink RSC-843

---
 gallery/src/components/NxAccordion/NxAccordionPage.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gallery/src/components/NxAccordion/NxAccordionPage.tsx b/gallery/src/components/NxAccordion/NxAccordionPage.tsx
index f4bff70911..5d0c761df7 100644
--- a/gallery/src/components/NxAccordion/NxAccordionPage.tsx
+++ b/gallery/src/components/NxAccordion/NxAccordionPage.tsx
@@ -44,7 +44,7 @@ const NxAccordionPage = () =>
       </NxP>
       <NxTile.Subsection>
         <NxTile.SubsectionHeader>
-          <NxH3>Guidelines</NxH3>
+          <NxH2>Guidelines</NxH2>
         </NxTile.SubsectionHeader>
         <NxH3>When to Use</NxH3>
         <NxList bulleted>
@@ -56,18 +56,18 @@ const NxAccordionPage = () =>
         <NxList bulleted>
           <NxList.Item>
             Displaying critical system information or a primary action to be taken on the page. (for example,
-            {' '}<a href="/components/alert">alerts</a>, confirmation or cancellation buttons).
+            {' '}<NxTextLink href="/components/alert">alerts</NxTextLink>, confirmation or cancellation buttons).
           </NxList.Item>
           <NxList.Item>
             Displaying navigation elements such as <a href="/components/tabs">tabs</a>.
           </NxList.Item>
           <NxList.Item>
             Displaying links pointing to sections of the same page, instead use a
-            {' '}<a href="/components/list">list</a>.
+            {' '}<NxTextLink href="/components/list">list</NxTextLink>.
           </NxList.Item>
           <NxList.Item>
             Creating hierarchy levels by nesting them within each other. If you need to add hierarchy to the
-            content use a <a href="/components/tree">tree</a>.
+            content use a <NxTextLink href="/components/tree">tree</NxTextLink>.
           </NxList.Item>
           <NxList.Item>
             Displaying a set of visual components following the same style, prefer using Collapsable.(TODO: Add