From 9cc941d3758e1b8ba83583c1af04f1923b6ac572 Mon Sep 17 00:00:00 2001 From: Navin Karkera Date: Fri, 5 Dec 2025 11:29:25 +0530 Subject: [PATCH 1/3] feat: add temporary message alert in sections settings tab in libraries Also increase sidebar width to remove `More` option and display all tabs together --- src/library-authoring/LibraryAuthoringPage.scss | 2 +- .../containers/ContainerInfo.tsx | 15 +++++++++++++-- src/library-authoring/containers/messages.ts | 5 +++++ 3 files changed, 19 insertions(+), 3 deletions(-) diff --git a/src/library-authoring/LibraryAuthoringPage.scss b/src/library-authoring/LibraryAuthoringPage.scss index 5a4470008c..5f0ceb7881 100644 --- a/src/library-authoring/LibraryAuthoringPage.scss +++ b/src/library-authoring/LibraryAuthoringPage.scss @@ -13,7 +13,7 @@ .library-authoring-sidebar { z-index: 1000; // same as header - flex: 500px 0 0; + flex: 530px 0 0; position: sticky; top: 0; right: 0; diff --git a/src/library-authoring/containers/ContainerInfo.tsx b/src/library-authoring/containers/ContainerInfo.tsx index d3d3afbd48..ed8843070e 100644 --- a/src/library-authoring/containers/ContainerInfo.tsx +++ b/src/library-authoring/containers/ContainerInfo.tsx @@ -8,10 +8,11 @@ import { Icon, IconButton, useToggle, + Alert, } from '@openedx/paragon'; import React, { useCallback } from 'react'; import { Link } from 'react-router-dom'; -import { MoreVert } from '@openedx/paragon/icons'; +import { InfoOutline, MoreVert } from '@openedx/paragon/icons'; import { useClipboard } from '@src/generic/clipboard'; import { ContainerType, getBlockType } from '@src/generic/key-utils'; @@ -149,6 +150,16 @@ const ContainerActions = ({ ); }; +/* istanbul ignore next */ +const ContainerSettings = () => { + const intl = useIntl(); + return ( + +

{intl.formatMessage(messages.containerSettingsMsg)}

+
+ ); +}; + const ContainerInfo = () => { const intl = useIntl(); const { @@ -222,7 +233,7 @@ const ContainerInfo = () => { {renderTab( CONTAINER_INFO_TABS.Settings, intl.formatMessage(messages.settingsTabTitle), - // TODO: container settings component + , )} diff --git a/src/library-authoring/containers/messages.ts b/src/library-authoring/containers/messages.ts index f18da2c389..ac0dfcd8ca 100644 --- a/src/library-authoring/containers/messages.ts +++ b/src/library-authoring/containers/messages.ts @@ -66,6 +66,11 @@ const messages = defineMessages({ defaultMessage: 'Container actions menu', description: 'Alt/title text for the container card menu button.', }, + containerSettingsMsg: { + id: 'course-authoring.library-authoring.container.settings.alert.message', + defaultMessage: 'Section settings cannot be configured with Libraries and must be set withing a course. In a future release, Libraries may support configuring some settings.', + description: 'Temporary message for settings tab being', + }, menuOpen: { id: 'course-authoring.library-authoring.menu.open', defaultMessage: 'Open', From d9357ef624b0d22458e1f6d30d2f99cb994653f9 Mon Sep 17 00:00:00 2001 From: Navin Karkera Date: Fri, 5 Dec 2025 19:35:52 +0530 Subject: [PATCH 2/3] fix: text --- src/library-authoring/containers/messages.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/library-authoring/containers/messages.ts b/src/library-authoring/containers/messages.ts index ac0dfcd8ca..b4cbbc6fdc 100644 --- a/src/library-authoring/containers/messages.ts +++ b/src/library-authoring/containers/messages.ts @@ -68,7 +68,7 @@ const messages = defineMessages({ }, containerSettingsMsg: { id: 'course-authoring.library-authoring.container.settings.alert.message', - defaultMessage: 'Section settings cannot be configured with Libraries and must be set withing a course. In a future release, Libraries may support configuring some settings.', + defaultMessage: 'Section settings cannot be configured within Libraries and must be set within a course. In a future release, Libraries may support configuring some settings.', description: 'Temporary message for settings tab being', }, menuOpen: { From 52ff9361ab732405275e6cd7a99ac168f625e3f1 Mon Sep 17 00:00:00 2001 From: Navin Karkera Date: Fri, 12 Dec 2025 11:33:36 +0530 Subject: [PATCH 3/3] refactor: apply suggestion src/library-authoring/containers/ContainerInfo.tsx MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Chris Chávez --- .../containers/ContainerInfo.tsx | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/src/library-authoring/containers/ContainerInfo.tsx b/src/library-authoring/containers/ContainerInfo.tsx index ed8843070e..d2c7c4c78c 100644 --- a/src/library-authoring/containers/ContainerInfo.tsx +++ b/src/library-authoring/containers/ContainerInfo.tsx @@ -151,15 +151,14 @@ const ContainerActions = ({ }; /* istanbul ignore next */ -const ContainerSettings = () => { - const intl = useIntl(); - return ( - -

{intl.formatMessage(messages.containerSettingsMsg)}

-
- ); -}; - +/* istanbul ignore next */ +const ContainerSettings = () => ( + +

+ +

+
+); const ContainerInfo = () => { const intl = useIntl(); const {