Skip to content

Commit

Permalink
feat: Removed course header stuff (#715)
Browse files Browse the repository at this point in the history
  • Loading branch information
asadiqbal08 authored Dec 20, 2021
1 parent b8875f3 commit 1546c62
Show file tree
Hide file tree
Showing 18 changed files with 34 additions and 325 deletions.
15 changes: 15 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@
"@edx/frontend-lib-special-exams": "1.14.1",
"@edx/frontend-platform": "1.14.3",
"@edx/paragon": "16.19.0",
"@edx/frontend-component-header": "^2.4.2",
"@fortawesome/fontawesome-svg-core": "1.2.36",
"@fortawesome/free-brands-svg-icons": "5.15.4",
"@fortawesome/free-regular-svg-icons": "5.15.4",
Expand Down
34 changes: 0 additions & 34 deletions src/course-header/AnonymousUserMenu.jsx

This file was deleted.

76 changes: 0 additions & 76 deletions src/course-header/AuthenticatedUserDropdown.jsx

This file was deleted.

100 changes: 0 additions & 100 deletions src/course-header/Header.jsx

This file was deleted.

29 changes: 0 additions & 29 deletions src/course-header/Header.test.jsx

This file was deleted.

46 changes: 0 additions & 46 deletions src/course-header/messages.js

This file was deleted.

2 changes: 1 addition & 1 deletion src/course-home/goal-unsubscribe/GoalUnsubscribe.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { useParams } from 'react-router-dom';
import { sendTrackEvent } from '@edx/frontend-platform/analytics';
import { injectIntl, intlShape } from '@edx/frontend-platform/i18n';

import { Header } from '../../course-header';
import { LearningHeader as Header } from '@edx/frontend-component-header';
import PageLoading from '../../generic/PageLoading';
import { unsubscribeFromCourseGoal } from '../data/api';

Expand Down
File renamed without changes.
2 changes: 1 addition & 1 deletion src/course-header/index.js → src/course-tabs/index.js
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
export { default as Header } from './Header';
/* eslint-disable import/prefer-default-export */
export { default as CourseTabsNavigation } from './CourseTabsNavigation';
11 changes: 11 additions & 0 deletions src/course-tabs/messages.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
import { defineMessages } from '@edx/frontend-platform/i18n';

const messages = defineMessages({
courseMaterial: {
id: 'learn.navigation.course.tabs.label',
defaultMessage: 'Course Material',
description: 'The accessible label for course tabs navigation',
},
});

export default messages;
2 changes: 1 addition & 1 deletion src/courseware/CoursewareContainer.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ describe('CoursewareContainer', () => {
const courseId = defaultCourseId;

function assertLoadedHeader(container) {
const courseHeader = container.querySelector('.course-header');
const courseHeader = container.querySelector('.learning-header');
// Ensure the course number and org appear - this proves we loaded course metadata properly.
expect(courseHeader).toHaveTextContent(courseMetadata.number);
expect(courseHeader).toHaveTextContent(courseMetadata.org);
Expand Down
10 changes: 0 additions & 10 deletions src/generic/messages.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,6 @@ const messages = defineMessages({
defaultMessage: 'register',
description: 'Text in a link, prompting the user to create an account. Used in "learning.logistration.alert"',
},
registerSentenceCase: {
id: 'general.register.sentenceCase',
defaultMessage: 'Register',
description: 'Text in a button, prompting the user to register.',
},
signInLowercase: {
id: 'learning.logistration.login', // ID left for historical purposes
defaultMessage: 'sign in',
description: 'Text in a link, prompting the user to log in. Used in "learning.logistration.alert"',
},
signInSentenceCase: {
id: 'general.signIn.sentenceCase',
defaultMessage: 'Sign in',
Expand Down
Loading

0 comments on commit 1546c62

Please sign in to comment.