@patternfly/chatbot 6.9.0-prerelease.4 → 6.9.0-prerelease.6

package/dist/cjs/ChatbotFooter/ChatbotFootnote.js

@@ -23,7 +23,7 @@ const ChatbotPopover_1 = __importDefault(require("../ChatbotPopover/ChatbotPopov
 const ChatbotFootnote = (_a) => {
     var _b, _c, _d, _e, _f;
     var { label, popover, className } = _a, props = __rest(_a, ["label", "popover", "className"]);
-    if (!(popover === null || popover === void 0 ? void 0 : popover.cta) && !(popover === null || popover === void 0 ? void 0 : popover.showClose)) {
+    if (popover && !popover.cta && !popover.showClose) {
         // eslint-disable-next-line no-console
         console.error('ChatbotFootnote: You must provide either the popover.cta or popover.showClose props in order to render a button that can close the popover.');
     }

package/dist/css/main.css

@@ -2424,11 +2424,11 @@ li[id*=user-content-fn-]:has(> span > span > .pf-chatbot__message-text + .pf-cha
   transition: border-color var(--pf-t--global--motion--timing-function--accelerate) var(--pf-t--global--motion--duration--sm), border-width var(--pf-t--global--motion--timing-function--accelerate) var(--pf-t--global--motion--duration--sm);
 }
 .pf-chatbot__message-bar:hover::after {
-  border-color: var(--pf-t--global--border--color--control--hover);
+  border-color: var(--pf-t--global--border--color--hover);
   border-width: var(--pf-t--global--border--width--control--hover);
 }
 .pf-chatbot__message-bar:focus-within::after {
-  border-color: var(--pf-t--global--border--color--control--clicked);
+  border-color: var(--pf-t--global--border--color--clicked);
   border-width: var(--pf-t--global--border--width--control--clicked);
 }
 .pf-chatbot__message-bar-actions {

package/dist/esm/ChatbotFooter/ChatbotFootnote.js

@@ -17,7 +17,7 @@ import ChatbotPopover from '../ChatbotPopover/ChatbotPopover';
 export const ChatbotFootnote = (_a) => {
     var _b, _c, _d, _e, _f;
     var { label, popover, className } = _a, props = __rest(_a, ["label", "popover", "className"]);
-    if (!(popover === null || popover === void 0 ? void 0 : popover.cta) && !(popover === null || popover === void 0 ? void 0 : popover.showClose)) {
+    if (popover && !popover.cta && !popover.showClose) {
         // eslint-disable-next-line no-console
         console.error('ChatbotFootnote: You must provide either the popover.cta or popover.showClose props in order to render a button that can close the popover.');
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/chatbot",
-  "version": "6.9.0-prerelease.4",
+  "version": "6.9.0-prerelease.6",
   "description": "This library provides React components based on PatternFly 6 that can be used to build chatbots.",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",

package/src/ChatbotFooter/ChatbotFooternote.test.tsx

@@ -1,6 +1,9 @@
 import { act, fireEvent, render, screen, waitFor } from '@testing-library/react';
 import ChatbotFootnote from './ChatbotFootnote';
 
+const POPOVER_CLOSE_ERROR =
+  'ChatbotFootnote: You must provide either the popover.cta or popover.showClose props in order to render a button that can close the popover.';
+
 describe('ChatbotFooternote', () => {
   const onClick = jest.fn();
   const popoverProps = {
@@ -20,11 +23,58 @@ describe('ChatbotFooternote', () => {
     }
   };
 
+  let consoleErrorSpy: jest.SpyInstance;
+
+  beforeEach(() => {
+    consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(jest.fn());
+  });
+
+  afterEach(() => {
+    consoleErrorSpy.mockRestore();
+  });
+
   it('should render ChatbotFooternote', () => {
     render(<ChatbotFootnote label="Chatbot footer" />);
     expect(screen.getByText('Chatbot footer')).toBeTruthy();
   });
 
+  it('should not log a console error when no popover is provided', () => {
+    render(<ChatbotFootnote label="Chatbot footer" />);
+    expect(consoleErrorSpy).not.toHaveBeenCalledWith(POPOVER_CLOSE_ERROR);
+  });
+
+  it('should not log a console error when popover has a cta', () => {
+    render(<ChatbotFootnote label="Chatbot footer" popover={popoverProps} />);
+    expect(consoleErrorSpy).not.toHaveBeenCalledWith(POPOVER_CLOSE_ERROR);
+  });
+
+  it('should not log a console error when popover has showClose', () => {
+    render(
+      <ChatbotFootnote
+        label="Chatbot footer"
+        popover={{
+          title: 'Verify accuracy',
+          description: 'description',
+          showClose: true
+        }}
+      />
+    );
+    expect(consoleErrorSpy).not.toHaveBeenCalledWith(POPOVER_CLOSE_ERROR);
+  });
+
+  it('should log a console error when popover is provided without cta or showClose', () => {
+    render(
+      <ChatbotFootnote
+        label="Chatbot footer"
+        popover={{
+          title: 'Verify accuracy',
+          description: 'description'
+        }}
+      />
+    );
+    expect(consoleErrorSpy).toHaveBeenCalledWith(POPOVER_CLOSE_ERROR);
+  });
+
   it('should render ChatbotFooternote with popover', async () => {
     render(<ChatbotFootnote label="Chatbot footer" popover={popoverProps} />);
 

package/src/ChatbotFooter/ChatbotFootnote.tsx

@@ -63,7 +63,7 @@ export const ChatbotFootnote: FunctionComponent<ChatbotFootnoteProps> = ({
   className,
   ...props
 }: ChatbotFootnoteProps) => {
-  if (!popover?.cta && !popover?.showClose) {
+  if (popover && !popover.cta && !popover.showClose) {
     // eslint-disable-next-line no-console
     console.error(
       'ChatbotFootnote: You must provide either the popover.cta or popover.showClose props in order to render a button that can close the popover.'

package/src/MessageBar/MessageBar.scss

@@ -46,14 +46,14 @@
 
   &:hover {
     &::after {
-      border-color: var(--pf-t--global--border--color--control--hover);
+      border-color: var(--pf-t--global--border--color--hover);
       border-width: var(--pf-t--global--border--width--control--hover);
     }
   }
 
   &:focus-within {
     &::after {
-      border-color: var(--pf-t--global--border--color--control--clicked);
+      border-color: var(--pf-t--global--border--color--clicked);
       border-width: var(--pf-t--global--border--width--control--clicked);
     }
   }