diff --git a/canvas_modules/common-canvas/assets/images/arrange_horizontally.svg b/canvas_modules/common-canvas/assets/images/arrange_horizontally.svg deleted file mode 100644 index 1ef38b7eca..0000000000 --- a/canvas_modules/common-canvas/assets/images/arrange_horizontally.svg +++ /dev/null @@ -1 +0,0 @@ - \ No newline at end of file diff --git a/canvas_modules/common-canvas/assets/images/arrange_vertically.svg b/canvas_modules/common-canvas/assets/images/arrange_vertically.svg deleted file mode 100644 index ac1b5228fd..0000000000 --- a/canvas_modules/common-canvas/assets/images/arrange_vertically.svg +++ /dev/null @@ -1 +0,0 @@ - \ No newline at end of file diff --git a/canvas_modules/common-canvas/assets/images/notification_counter_icon.svg b/canvas_modules/common-canvas/assets/images/notification_counter_icon.svg index a250f462bc..b45a724677 100644 --- a/canvas_modules/common-canvas/assets/images/notification_counter_icon.svg +++ b/canvas_modules/common-canvas/assets/images/notification_counter_icon.svg @@ -1,3 +1,3 @@ - + diff --git a/canvas_modules/common-canvas/src/common-canvas/cc-contents.jsx b/canvas_modules/common-canvas/src/common-canvas/cc-contents.jsx index 8dc0953913..739e1ab446 100644 --- a/canvas_modules/common-canvas/src/common-canvas/cc-contents.jsx +++ b/canvas_modules/common-canvas/src/common-canvas/cc-contents.jsx @@ -479,6 +479,7 @@ class CanvasContents extends React.Component { onMouseDown={this.onMouseDown} onMouseLeave={this.onMouseLeave} onBlur={this.onBlur} onKeyDown={this.onKeyDown} onKeyUp={this.onKeyUp} + role="application" aria-label="canvas-keyboard-navigation" // Resolve Accessibility Violation of role and label /> ); } diff --git a/canvas_modules/common-canvas/src/icons/icon.jsx b/canvas_modules/common-canvas/src/icons/icon.jsx index 26c0b20e58..83ee080589 100644 --- a/canvas_modules/common-canvas/src/icons/icon.jsx +++ b/canvas_modules/common-canvas/src/icons/icon.jsx @@ -124,7 +124,7 @@ export default class Icon extends React.Component { icon = this.getCanvasIcon(this.props.type); if (typeof icon !== "undefined") { const className = classNames("canvas-icon", iconClassName, this.props.className); - icon = ; + icon = ; } else { icon =
; } 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 e13b580ee5..c9b19a8dec 100644 --- a/canvas_modules/common-canvas/src/toolbar/toolbar-button-item.jsx +++ b/canvas_modules/common-canvas/src/toolbar/toolbar-button-item.jsx @@ -18,8 +18,6 @@ import React from "react"; import PropTypes from "prop-types"; import Tooltip from "../tooltip/tooltip.jsx"; -import ArrangeHorizontally from "./../../assets/images/arrange_horizontally.svg"; -import ArrangeVertically from "./../../assets/images/arrange_vertically.svg"; import ToggleNotificationPanel from "./../../assets/images/notification_counter_icon.svg"; import { Button } from "@carbon/react"; @@ -29,7 +27,7 @@ import { StopFilledAlt, Play, Undo, Redo, Chat, ChatOff, Result, Cut, Copy, Paste, Edit, ColorPalette, Maximize, Minimize, Launch, AddComment, TrashCan, ZoomIn, ZoomOut, Checkmark, ChevronRight, ChevronDown, ChevronUp, - CenterToFit, OpenPanelFilledLeft } from "@carbon/react/icons"; + CenterToFit, OpenPanelFilledLeft, ArrangeVertical, ArrangeHorizontal } from "@carbon/react/icons"; import { TOOLBAR_STOP, TOOLBAR_RUN, TOOLBAR_UNDO, TOOLBAR_REDO, TOOLBAR_CUT, TOOLBAR_COPY, TOOLBAR_PASTE, TOOLBAR_CLIPBOARD, TOOLBAR_CREATE_COMMENT, TOOLBAR_CREATE_AUTO_COMMENT, @@ -123,12 +121,10 @@ class ToolbarButtonItem extends React.Component { return ; case (TOOLBAR_TOGGLE_PALETTE): return ; - - // Non-carbon icons case (TOOLBAR_ARRANGE_HORIZONALLY): - return ; + return ; case (TOOLBAR_ARRANGE_VERTICALLY): - return ; + return ; case (TOOLBAR_TOGGLE_NOTIFICATION_PANEL): return ;