From 3c981431cc8d84618bf925ac2d454fbc220c6e12 Mon Sep 17 00:00:00 2001 From: Rebecca Alpert Date: Tue, 13 Oct 2026 11:41:25 -0400 Subject: [PATCH 1/2] fix(ChatbotToggle): Fix non-round toggles We had a regression where non-round toggles were round. This should fix it. Fixes https://github.com/patternfly/chatbot/issues/923 --- packages/module/src/ChatbotToggle/ChatbotToggle.scss | 5 +++-- packages/module/src/ChatbotToggle/ChatbotToggle.test.tsx | 4 ++++ 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/module/src/ChatbotToggle/ChatbotToggle.scss b/packages/module/src/ChatbotToggle/ChatbotToggle.scss index eea4a1613..b56042e4d 100644 --- a/packages/module/src/ChatbotToggle/ChatbotToggle.scss +++ b/packages/module/src/ChatbotToggle/ChatbotToggle.scss @@ -5,6 +5,7 @@ position: fixed; inset-block-end: var(--pf-t--global--spacer--md); inset-inline-end: var(--pf-t--global--spacer--md); + --pf-v6-c-button--BorderRadius: unset; --pf-v6-c-button--m-plain--BackgroundColor: var(--pf-t--global--background--color--floating--default); --pf-v6-c-button--m-plain--hover--BackgroundColor: var(--pf-t--global--background--color--floating--hover); --pf-v6-c-button--m-plain--m-clicked--BackgroundColor: var(--pf-t--global--background--color--floating--clicked); @@ -48,8 +49,8 @@ } } -.pf-chatbot__button--round { - border-radius: var(--pf-t--global--border--radius--pill); +.pf-chatbot__button.pf-chatbot__button--round { + --pf-v6-c-button--BorderRadius: var(--pf-t--global--border--radius--pill); width: 3rem; height: 3rem; } diff --git a/packages/module/src/ChatbotToggle/ChatbotToggle.test.tsx b/packages/module/src/ChatbotToggle/ChatbotToggle.test.tsx index 3267f021f..e68229888 100644 --- a/packages/module/src/ChatbotToggle/ChatbotToggle.test.tsx +++ b/packages/module/src/ChatbotToggle/ChatbotToggle.test.tsx @@ -38,6 +38,10 @@ describe('ChatbotToggle', () => { expect(screen.getByRole('button')).toHaveClass('pf-chatbot__button'); expect(screen.getByRole('button')).toHaveClass('pf-chatbot__button--round'); }); + it('should handle isRound correctly when false', () => { + render(); + expect(screen.getByRole('button')).not.toHaveClass('pf-chatbot__button--round'); + }); it('should handle className correctly', () => { render(); expect(screen.getByRole('button')).toHaveClass('pf-chatbot__button'); From f086307e624ba7703dd15560c4d0b5e96c2e3632 Mon Sep 17 00:00:00 2001 From: Rebecca Alpert Date: Tue, 13 Oct 2026 13:30:35 -0400 Subject: [PATCH 2/2] Address feedback --- packages/module/src/ChatbotToggle/ChatbotToggle.scss | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/module/src/ChatbotToggle/ChatbotToggle.scss b/packages/module/src/ChatbotToggle/ChatbotToggle.scss index b56042e4d..e0158867f 100644 --- a/packages/module/src/ChatbotToggle/ChatbotToggle.scss +++ b/packages/module/src/ChatbotToggle/ChatbotToggle.scss @@ -5,7 +5,7 @@ position: fixed; inset-block-end: var(--pf-t--global--spacer--md); inset-inline-end: var(--pf-t--global--spacer--md); - --pf-v6-c-button--BorderRadius: unset; + --pf-v6-c-button--BorderRadius: 0; --pf-v6-c-button--m-plain--BackgroundColor: var(--pf-t--global--background--color--floating--default); --pf-v6-c-button--m-plain--hover--BackgroundColor: var(--pf-t--global--background--color--floating--hover); --pf-v6-c-button--m-plain--m-clicked--BackgroundColor: var(--pf-t--global--background--color--floating--clicked);