fix: fixed width issue of incontext sidebar (#1395)

openedx · May 27, 2024 · 1264b42 · 1264b42
1 parent e3ecee1
commit 1264b42
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/courseware/course/new-sidebar/common/SidebarBase.jsx b/src/courseware/course/new-sidebar/common/SidebarBase.jsx
@@ -104,7 +104,7 @@ SidebarBase.propTypes = {
 
 SidebarBase.defaultProps = {
   title: '',
-  width: '50rem',
+  width: '45rem',
   allowFullHeight: false,
   showTitleBar: true,
   className: '',

diff --git a/src/courseware/course/sidebar/common/SidebarBase.jsx b/src/courseware/course/sidebar/common/SidebarBase.jsx
@@ -42,7 +42,7 @@ const SidebarBase = ({
         'd-none': currentSidebar !== sidebarId,
       }, className)}
       data-testid={`sidebar-${sidebarId}`}
-      style={{ minWidth: shouldDisplayFullScreen ? '100%' : width }}
+      style={{ width: shouldDisplayFullScreen ? '100%' : width }}
       aria-label={ariaLabel}
       id="course-sidebar"
     >
@@ -98,7 +98,7 @@ SidebarBase.propTypes = {
 };
 
 SidebarBase.defaultProps = {
-  width: '410px',
+  width: '31rem',
   showTitleBar: true,
 };
 

diff --git a/src/courseware/course/sidebar/sidebars/notifications/NotificationTray.jsx b/src/courseware/course/sidebar/sidebars/notifications/NotificationTray.jsx
@@ -71,6 +71,7 @@ const NotificationTray = ({ intl }) => {
       title={intl.formatMessage(messages.notificationTitle)}
       ariaLabel={intl.formatMessage(messages.notificationTray)}
       sidebarId={ID}
+      width="45rem"
       className={classNames({
         'h-100': !verifiedMode && !shouldDisplayFullScreen,
         'ml-4': !shouldDisplayFullScreen,