fix(ui-tooltip,ui-popover): put aria-expanded on trigger element

INSTUI-4544

Author: HerrTopi

packages/ui-popover/src/Popover/index.tsx

@@ -437,7 +437,7 @@ class Popover extends Component<PopoverProps, PopoverState> {
     let trigger = this._renderTrigger
 
     if (trigger) {
-      const { on, shouldContainFocus } = this.props
+      const { on } = this.props
 
       let onClick: React.MouseEventHandler | undefined = undefined
       let onFocus: React.FocusEventHandler | undefined = undefined
@@ -468,17 +468,13 @@ class Popover extends Component<PopoverProps, PopoverState> {
         }
       }
 
-      if (shouldContainFocus) {
-        // only set aria-expanded if popover can contain focus
-        expanded = this.shown ? 'true' : 'false'
+      // only set aria-expanded if popover can contain focus
+      expanded = this.shown ? 'true' : 'false'
 
-        if ('aria-expanded' in this.props) {
-          // @ts-expect-error It is an escape hatch, in case someone
-          // wants to remove/override aria-expanded even when shouldContainFocus
-          expanded = this.props['aria-expanded']
-        }
-      } else {
-        expanded = undefined
+      if ('aria-expanded' in this.props) {
+        // @ts-expect-error It is an escape hatch, in case someone
+        // wants to remove/override aria-expanded even when shouldContainFocus
+        expanded = this.props['aria-expanded']
       }
 
       trigger = safeCloneElement(trigger, {

packages/ui-tooltip/src/Tooltip/index.tsx

@@ -177,6 +177,7 @@ class Tooltip extends Component<TooltipProps, TooltipState> {
         elementRef={this.handleRef}
         shouldCloseOnDocumentClick={false}
         shouldCloseOnEscape
+        aria-expanded={undefined}
       >
         {!preventTooltip ? (
           <span id={this._id} css={styles?.tooltip} role="tooltip">