Address additional feedback

patternfly · Nov 6, 2024 · 0995420 · 0995420
1 parent f92cebd
commit 0995420
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/...es/module/patternfly-docs/content/extensions/virtual-assistant/examples/demos/Chatbot.tsx b/...es/module/patternfly-docs/content/extensions/virtual-assistant/examples/demos/Chatbot.tsx
@@ -286,9 +286,12 @@ export const ChatbotDemo: React.FunctionComponent = () => {
   const handleSkipToContent = (e) => {
     e.preventDefault();
     if (displayMode === ChatbotDisplayMode.default) {
-      if (toggleRef.current) {
+      if (!chatbotVisible && toggleRef.current) {
         toggleRef.current.focus();
       }
+      if (chatbotVisible && chatbotRef.current) {
+        chatbotRef.current.focus();
+      }
     } else {
       if (chatbotRef.current) {
         chatbotRef.current.focus();

diff --git a/packages/module/src/Chatbot/Chatbot.tsx b/packages/module/src/Chatbot/Chatbot.tsx
@@ -58,15 +58,14 @@ const ChatbotBase: React.FunctionComponent<ChatbotProps> = ({
       {/* Ref is intended for use with skip to chatbot links, etc. */}
       {/* Motion.div does not accept refs */}
       {isVisible ? (
-        <div
-          role="region"
+        <section
           aria-label={props['aria-label'] ?? 'Chatbot'}
           className={`pf-chatbot-container pf-chatbot-container--${displayMode} ${!isVisible ? 'pf-chatbot-container--hidden' : ''}`}
-          tabIndex={0}
+          tabIndex={-1}
           ref={innerRef}
         >
           {children}
-        </div>
+        </section>
       ) : undefined}
     </motion.div>
   );