From 28e7cc08ebf448fd7824a752f719a541cb05e017 Mon Sep 17 00:00:00 2001
From: PRINCESANCHEZ <67511950+PRINCESANCHEZ@users.noreply.github.com>
Date: Thu, 17 Oct 2024 10:54:19 -0700
Subject: [PATCH 1/4] #2211 Fix Accessibility Issues for palette (#2212)

---
 .../common-canvas/src/palette/palette-content-list-item.jsx     | 2 +-
 .../src/palette/palette-flyout-content-category.jsx             | 2 +-
 .../common-canvas/src/palette/palette-flyout-content-search.jsx | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/canvas_modules/common-canvas/src/palette/palette-content-list-item.jsx b/canvas_modules/common-canvas/src/palette/palette-content-list-item.jsx
index 66ae514556..b0587f5505 100644
--- a/canvas_modules/common-canvas/src/palette/palette-content-list-item.jsx
+++ b/canvas_modules/common-canvas/src/palette/palette-content-list-item.jsx
@@ -284,7 +284,7 @@ class PaletteContentListItem extends React.Component {
 				icon = image;
 			} else if (typeof image === "string") {
 				icon = image.endsWith(".svg")
-					? <SVG src={image} className="palette-list-item-icon" draggable="false" />
+					? <SVG src={image} className="palette-list-item-icon" draggable="false" aria-hidden="true" />
 					: <img src={image} className="palette-list-item-icon" draggable="false" alt={""} />;
 			}
 		}
diff --git a/canvas_modules/common-canvas/src/palette/palette-flyout-content-category.jsx b/canvas_modules/common-canvas/src/palette/palette-flyout-content-category.jsx
index 4476c30bec..ef9ac5fbd9 100644
--- a/canvas_modules/common-canvas/src/palette/palette-flyout-content-category.jsx
+++ b/canvas_modules/common-canvas/src/palette/palette-flyout-content-category.jsx
@@ -170,7 +170,7 @@ class PaletteFlyoutContentCategory extends React.Component {
 			} else if (this.props.category.image.endsWith(".svg")) {
 				itemImage = (
 					<div>
-						<SVG src={this.props.category.image} className="palette-flyout-category-item-icon" draggable="false" />
+						<SVG src={this.props.category.image} className="palette-flyout-category-item-icon" draggable="false" aria-hidden="true" />
 					</div>
 				);
 			} else {
diff --git a/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx b/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx
index fc3157aa53..601b8772c0 100644
--- a/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx
+++ b/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx
@@ -52,7 +52,7 @@ class PaletteFlyoutContentSearch extends React.Component {
 			// palette-flyout-search id added for hopscotch tours
 			<div className="palette-flyout-search-container" id="palette-flyout-search">
 				<Search
-					id="palette-flyout-search"
+					id="palette-flyout-search-input"
 					className="palette-flyout-search"
 					placeholder={placeHolder}
 					onChange={this.handleSearchStringChange}

From d8bf1c367f22be586be02eef457aff55b1ae2364 Mon Sep 17 00:00:00 2001
From: PRINCESANCHEZ <67511950+PRINCESANCHEZ@users.noreply.github.com>
Date: Fri, 18 Oct 2024 11:40:08 -0700
Subject: [PATCH 2/4] #2215 Fix Accessibility Issues for palette search (#2216)

Signed-off-by: Prince Sanchez <prince.sanchez@ibm.com>
---
 .../common-canvas/src/palette/palette-flyout-content-search.jsx  | 1 -
 1 file changed, 1 deletion(-)

diff --git a/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx b/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx
index 601b8772c0..b8893f3aaa 100644
--- a/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx
+++ b/canvas_modules/common-canvas/src/palette/palette-flyout-content-search.jsx
@@ -52,7 +52,6 @@ class PaletteFlyoutContentSearch extends React.Component {
 			// palette-flyout-search id added for hopscotch tours
 			<div className="palette-flyout-search-container" id="palette-flyout-search">
 				<Search
-					id="palette-flyout-search-input"
 					className="palette-flyout-search"
 					placeholder={placeHolder}
 					onChange={this.handleSearchStringChange}

From 0fe97d7bec14678592021d32e2a85dfd7911cde4 Mon Sep 17 00:00:00 2001
From: PRINCESANCHEZ <67511950+PRINCESANCHEZ@users.noreply.github.com>
Date: Mon, 21 Oct 2024 12:13:19 -0700
Subject: [PATCH 3/4] #2218 Fix accessibility violations (#2219)

Signed-off-by: Prince Sanchez <prince.sanchez@ibm.com>
---
 .../common-properties/components/title-editor/title-editor.jsx  | 2 +-
 .../common-canvas/src/toolbar/toolbar-button-item.jsx           | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/canvas_modules/common-canvas/src/common-properties/components/title-editor/title-editor.jsx b/canvas_modules/common-canvas/src/common-properties/components/title-editor/title-editor.jsx
index d6be07888c..952a0b699c 100644
--- a/canvas_modules/common-canvas/src/common-properties/components/title-editor/title-editor.jsx
+++ b/canvas_modules/common-canvas/src/common-properties/components/title-editor/title-editor.jsx
@@ -174,7 +174,7 @@ class TitleEditor extends Component {
 				</div>)
 				: null;
 			const icon = this.props.icon && typeof this.props.icon === "string"
-				? <Isvg className="properties-title-heading-icon" src={this.props.icon} />
+				? <Isvg className="properties-title-heading-icon" src={this.props.icon} aria-hidden="true" />
 				: null;
 			if (label || icon) {
 				heading = (<div className="properties-title-heading">
diff --git a/canvas_modules/common-canvas/src/toolbar/toolbar-button-item.jsx b/canvas_modules/common-canvas/src/toolbar/toolbar-button-item.jsx
index b105a4c2f8..e13b580ee5 100644
--- a/canvas_modules/common-canvas/src/toolbar/toolbar-button-item.jsx
+++ b/canvas_modules/common-canvas/src/toolbar/toolbar-button-item.jsx
@@ -275,7 +275,7 @@ class ToolbarButtonItem extends React.Component {
 			}
 			const path = this.props.size === "sm" ? "M 29 29 L 29 23 23 29 Z" : "M 37 37 L 37 30 30 37 Z";
 			return (
-				<svg className="toolbar-tick-svg">
+				<svg className="toolbar-tick-svg" aria-hidden="true">
 					<path d={path} className="toolbar-tick-mark" />
 				</svg>
 			);

From 89e8d63d04a6563ab25f80c54ced866e3c49ee61 Mon Sep 17 00:00:00 2001
From: mikieyx <103902194+mikieyx@users.noreply.github.com>
Date: Wed, 23 Oct 2024 10:58:57 -0700
Subject: [PATCH 4/4] #2220 SubAreas sample toolbar - 'Save' button should have
 a label  (#2221)

Signed-off-by: Michael Pavlik <michael@michaels-mbp.usca.ibm.com>
---
 canvas_modules/harness/src/client/App.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/canvas_modules/harness/src/client/App.js b/canvas_modules/harness/src/client/App.js
index 308471e827..310db75a51 100644
--- a/canvas_modules/harness/src/client/App.js
+++ b/canvas_modules/harness/src/client/App.js
@@ -2225,7 +2225,7 @@ class App extends React.Component {
 					{ action: "color-subpanel", iconEnabled: (<ColorPalette size={32} />), label: "Color picker", enable: true,
 						subPanel: ColorPicker, subPanelData: { clickActionHandler: (color) => window.alert("Color selected = " + color) } },
 					{ divider: true },
-					{ action: "save", iconEnabled: (<Save size={32} />), enable: true, tooltip: saveReloadTooltip }
+					{ action: "save", iconEnabled: (<Save size={32} />), label: "Save", enable: true, tooltip: saveReloadTooltip }
 				],
 				rightBar: [
 					{ divider: true },