From f2f4f5f3a55b5b34f66b48d7983a5b31b01eafd4 Mon Sep 17 00:00:00 2001 From: julianajlk Date: Wed, 14 Jul 2021 12:25:32 -0400 Subject: [PATCH] Fix bug that was applying the CSS to other active classes (#543) --- src/courseware/course/Course.test.jsx | 4 ++-- src/courseware/course/NotificationTrigger.jsx | 2 +- src/courseware/course/NotificationTrigger.scss | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/courseware/course/Course.test.jsx b/src/courseware/course/Course.test.jsx index d825d7c6..dd4a7785 100644 --- a/src/courseware/course/Course.test.jsx +++ b/src/courseware/course/Course.test.jsx @@ -94,9 +94,9 @@ describe('Course', () => { const notificationTrigger = screen.getByRole('button', { name: /Show notification tray/i }); expect(notificationTrigger).toBeInTheDocument(); - expect(notificationTrigger).toHaveClass('active'); + expect(notificationTrigger).toHaveClass('trigger-active'); fireEvent.click(notificationTrigger); - expect(notificationTrigger).not.toHaveClass('active'); + expect(notificationTrigger).not.toHaveClass('trigger-active'); }); it('passes handlers to the sequence', async () => { diff --git a/src/courseware/course/NotificationTrigger.jsx b/src/courseware/course/NotificationTrigger.jsx index 183a35a1..3486ae4e 100644 --- a/src/courseware/course/NotificationTrigger.jsx +++ b/src/courseware/course/NotificationTrigger.jsx @@ -9,7 +9,7 @@ import messages from './messages'; function NotificationTrigger({ intl, toggleNotificationTray, isNotificationTrayVisible }) { return (