Skip to content

Commit

Permalink
chore: address PR review
Browse files Browse the repository at this point in the history
  • Loading branch information
BryanttV committed Jan 4, 2024
1 parent 99da5b8 commit 0f9a05b
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/course-home/outline-tab/Section.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ const Section = ({
{hideFromTOC && (
<span className="small d-flex align-content-end">

Check warning on line 75 in src/course-home/outline-tab/Section.jsx

View check run for this annotation

Codecov / codecov/patch

src/course-home/outline-tab/Section.jsx#L75

Added line #L75 was not covered by tests
<Icon className="mr-2" src={DisabledVisible} />
Hidden in Course Outline, accessible via link
{intl.formatMessage(messages.hiddenSection)}
</span>
)}
</div>
Expand Down
3 changes: 2 additions & 1 deletion src/course-home/outline-tab/SequenceLink.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,8 @@ const SequenceLink = ({
<div className="row w-100 my-2 mx-4 pl-3">
{hideFromTOC && (
<span className="small d-flex">

Check warning on line 122 in src/course-home/outline-tab/SequenceLink.jsx

View check run for this annotation

Codecov / codecov/patch

src/course-home/outline-tab/SequenceLink.jsx#L122

Added line #L122 was not covered by tests
<Icon className="mr-2" src={DisabledVisible} />Subsections are not navigable beetwen each other, they can only be accessed through their link.
<Icon className="mr-2" src={DisabledVisible} />
{intl.formatMessage(messages.hiddenSequenceLink)}
</span>
)}
</div>
Expand Down
10 changes: 10 additions & 0 deletions src/course-home/outline-tab/messages.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,16 @@ const messages = defineMessages({
defaultMessage: 'Completed section',
description: 'Text used to describe the green checkmark icon in front of a section title',
},
hiddenSection: {
id: 'learning.outline.hiddenSection',
defaultMessage: 'Hidden in Course Outline, accessible via link.',
description: 'Label for hidden section in course outline',
},
hiddenSequenceLink: {
id: 'learning.outline.hiddenSequenceLink',
defaultMessage: 'Subsections are not navigable beetwen each other, they can only be accessed through their link.',
description: 'Label for hidden sequence in course outline',
},
dates: {
id: 'learning.outline.dates',
defaultMessage: 'Important dates',
Expand Down

0 comments on commit 0f9a05b

Please sign in to comment.