Merge pull request #228 from rebeccaalpert/a11y-214

fix(Chatbot/JumpButton): Remove content from DOM when not visible

Author: nicolethoen

packages/module/src/Chatbot/Chatbot.tsx

@@ -50,7 +50,7 @@ export const Chatbot: React.FunctionComponent<ChatbotProps> = ({
       initial="hidden"
       animate={isVisible ? 'visible' : 'hidden'}
     >
-      {children}
+      {isVisible ? children : undefined}
     </motion.div>
   );
 };

packages/module/src/MessageBox/JumpButton.tsx

@@ -18,20 +18,21 @@ export interface JumpButtonProps {
   isHidden?: boolean;
 }
 
-const JumpButton: React.FunctionComponent<JumpButtonProps> = ({ position, isHidden, onClick }: JumpButtonProps) => (
-  <Tooltip id={`pf-chatbot__tooltip--jump-${position}`} content={`Back to ${position}`} position="top">
-    <Button
-      variant="plain"
-      className={`pf-chatbot__jump pf-chatbot__jump--${position} ${isHidden && `pf-chatbot__jump--${position}--hidden`}`}
-      aria-label={`Jump ${position} button`}
-      aria-describedby={`pf-chatbot__tooltip--jump-${position}`}
-      onClick={onClick}
-    >
-      <Icon iconSize="xl" isInline>
-        {position === 'top' ? <ArrowUpIcon /> : <ArrowDownIcon />}
-      </Icon>
-    </Button>
-  </Tooltip>
-);
+const JumpButton: React.FunctionComponent<JumpButtonProps> = ({ position, isHidden, onClick }: JumpButtonProps) =>
+  isHidden ? undefined : (
+    <Tooltip id={`pf-chatbot__tooltip--jump-${position}`} content={`Back to ${position}`} position="top">
+      <Button
+        variant="plain"
+        className={`pf-chatbot__jump pf-chatbot__jump--${position} ${isHidden && `pf-chatbot__jump--${position}--hidden`}`}
+        aria-label={`Jump ${position} button`}
+        aria-describedby={`pf-chatbot__tooltip--jump-${position}`}
+        onClick={onClick}
+      >
+        <Icon iconSize="xl" isInline>
+          {position === 'top' ? <ArrowUpIcon /> : <ArrowDownIcon />}
+        </Icon>
+      </Button>
+    </Tooltip>
+  );
 
 export default JumpButton;