From 08c1c6df69bd46432df306d2e7106a663092f24b Mon Sep 17 00:00:00 2001
From: Kimberly Hart <103072652+khartco@users.noreply.github.com>
Date: Tue, 14 Jan 2025 16:29:20 -0700
Subject: [PATCH] [DEV-9281] Bar Charts: labels and outline enabled by default,
 add tooltips recommending to remain enabled

---
 .../components/Panels/Panel.General.tsx           |  4 ++++
 .../components/Panels/Panel.Visual.tsx            | 15 +++++++++++++++
 packages/chart/src/data/initial-state.js          |  4 ++--
 3 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/packages/chart/src/components/EditorPanel/components/Panels/Panel.General.tsx b/packages/chart/src/components/EditorPanel/components/Panels/Panel.General.tsx
index a71bbd0c0..4d8ebcc82 100644
--- a/packages/chart/src/components/EditorPanel/components/Panels/Panel.General.tsx
+++ b/packages/chart/src/components/EditorPanel/components/Panels/Panel.General.tsx
@@ -209,6 +209,10 @@ const PanelGeneral: FC<PanelProps> = props => {
                     <Icon display='question' style={{ marginLeft: '0.5rem' }} />
                   </Tooltip.Target>
                   <Tooltip.Content>
+                    <p>
+                      Recommended set to display for Section 508 compliance.
+                    </p>
+                    <hr/>
                     <p>
                       Selecting this option will <i> not </i> hide the display of "zero value", "suppressed data", or
                       "missing data" indicators on the chart (if applicable).
diff --git a/packages/chart/src/components/EditorPanel/components/Panels/Panel.Visual.tsx b/packages/chart/src/components/EditorPanel/components/Panels/Panel.Visual.tsx
index 8eb3a3872..099c1a3ea 100644
--- a/packages/chart/src/components/EditorPanel/components/Panels/Panel.Visual.tsx
+++ b/packages/chart/src/components/EditorPanel/components/Panels/Panel.Visual.tsx
@@ -138,6 +138,21 @@ const PanelVisual: FC<PanelProps> = props => {
         )}
         {visHasBarBorders() && (
           <Select
+            tooltip={
+              <Tooltip style={{ textTransform: 'none' }}>
+                <Tooltip.Target>
+                  <Icon
+                    display='question'
+                    style={{ marginLeft: '0.5rem', display: 'inline-block', whiteSpace: 'nowrap' }}
+                  />
+                </Tooltip.Target>
+                <Tooltip.Content>
+                  <p>
+                  Recommended set to display for Section 508 compliance.
+                  </p>
+                </Tooltip.Content>
+              </Tooltip>
+            }
             value={config.barHasBorder}
             fieldName='barHasBorder'
             label='Bar Borders'
diff --git a/packages/chart/src/data/initial-state.js b/packages/chart/src/data/initial-state.js
index e3ffa922b..dc6a53eed 100644
--- a/packages/chart/src/data/initial-state.js
+++ b/packages/chart/src/data/initial-state.js
@@ -13,7 +13,7 @@ export default {
   animate: false,
   lineDatapointStyle: 'hover',
   lineDatapointColor: 'Same as Line',
-  barHasBorder: 'false',
+  barHasBorder: 'true',
   isLollipopChart: false,
   lollipopShape: 'circle',
   lollipopColorStyle: 'two-tone',
@@ -188,7 +188,7 @@ export default {
     palette: 'monochrome-1',
     isPaletteReversed: false
   },
-  labels: false,
+  labels: true,
   dataFormat: {
     commas: false,
     prefix: '',