Skip to content

Commit fedb855

Browse files
authored
feat: add temporary message alert in sections settings tab in libraries (#2734) (#2766)
- add temporary message alert in sections settings tab in libraries - increase sidebar width to remove `More` option and display all tabs together (cherry picked from commit 3eeca24)
1 parent 18e51db commit fedb855

3 files changed

Lines changed: 18 additions & 3 deletions

File tree

src/library-authoring/LibraryAuthoringPage.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313

1414
.library-authoring-sidebar {
1515
z-index: 1000; // same as header
16-
flex: 500px 0 0;
16+
flex: 530px 0 0;
1717
position: sticky;
1818
top: 0;
1919
right: 0;

src/library-authoring/containers/ContainerInfo.tsx

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,11 @@ import {
88
Icon,
99
IconButton,
1010
useToggle,
11+
Alert,
1112
} from '@openedx/paragon';
1213
import React, { useCallback } from 'react';
1314
import { Link } from 'react-router-dom';
14-
import { MoreVert } from '@openedx/paragon/icons';
15+
import { InfoOutline, MoreVert } from '@openedx/paragon/icons';
1516

1617
import { useClipboard } from '@src/generic/clipboard';
1718
import { ContainerType, getBlockType } from '@src/generic/key-utils';
@@ -149,6 +150,15 @@ const ContainerActions = ({
149150
);
150151
};
151152

153+
/* istanbul ignore next */
154+
/* istanbul ignore next */
155+
const ContainerSettings = () => (
156+
<Alert icon={InfoOutline} variant="info">
157+
<p>
158+
<FormattedMessage {...messages.containerSettingsMsg} />
159+
</p>
160+
</Alert>
161+
);
152162
const ContainerInfo = () => {
153163
const intl = useIntl();
154164
const {
@@ -222,7 +232,7 @@ const ContainerInfo = () => {
222232
{renderTab(
223233
CONTAINER_INFO_TABS.Settings,
224234
intl.formatMessage(messages.settingsTabTitle),
225-
// TODO: container settings component
235+
<ContainerSettings />,
226236
)}
227237
</Tabs>
228238
</Stack>

src/library-authoring/containers/messages.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,11 @@ const messages = defineMessages({
6666
defaultMessage: 'Container actions menu',
6767
description: 'Alt/title text for the container card menu button.',
6868
},
69+
containerSettingsMsg: {
70+
id: 'course-authoring.library-authoring.container.settings.alert.message',
71+
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.',
72+
description: 'Temporary message for settings tab being',
73+
},
6974
menuOpen: {
7075
id: 'course-authoring.library-authoring.menu.open',
7176
defaultMessage: 'Open',

0 commit comments

Comments
 (0)