diff --git a/kolibri/plugins/coach/assets/src/constants/tabsConstants.js b/kolibri/plugins/coach/assets/src/constants/tabsConstants.js index 44e7fb9ca57..baedbde1b1d 100644 --- a/kolibri/plugins/coach/assets/src/constants/tabsConstants.js +++ b/kolibri/plugins/coach/assets/src/constants/tabsConstants.js @@ -20,8 +20,8 @@ export const ReportsGroupTabs = { ACTIVITY: 'tabActivity', }; -export const LEARNERS_TABS_ID = 'reportLearners'; -export const LearnersTabs = { +export const REPORTS_LEARNERS_TABS_ID = 'coachReportsLearners'; +export const ReportsLearnersTabs = { REPORTS: 'tabReports', ACTIVITY: 'tabActivity', }; diff --git a/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerActivityPage.vue b/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerActivityPage.vue index 6fd5734df9d..ed5442ccfba 100644 --- a/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerActivityPage.vue +++ b/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerActivityPage.vue @@ -8,10 +8,10 @@ - + @@ -71,7 +71,7 @@ import commonCoreStrings from 'kolibri.coreVue.mixins.commonCoreStrings'; import commonCoach from '../common'; - import { LEARNERS_TABS_ID, LearnersTabs } from '../../constants/tabsConstants'; + import { REPORTS_LEARNERS_TABS_ID, ReportsLearnersTabs } from '../../constants/tabsConstants'; import { useCoachTabs } from '../../composables/useCoachTabs'; export default { @@ -97,7 +97,7 @@ }, data() { return { - LEARNERS_TABS_ID, + REPORTS_LEARNERS_TABS_ID, }; }, computed: { @@ -136,12 +136,12 @@ tabs() { return [ { - id: LearnersTabs.REPORTS, + id: ReportsLearnersTabs.REPORTS, label: this.coachString('reportsLabel'), to: this.classRoute('ReportsLearnerReportPage', {}), }, { - id: LearnersTabs.ACTIVITY, + id: ReportsLearnersTabs.ACTIVITY, label: this.coachString('activityLabel'), to: this.classRoute('ReportsLearnerActivityPage', {}), }, @@ -153,8 +153,8 @@ // that this header was re-mounted as a result // of navigation after clicking a tab (focus shouldn't // be manipulated programatically in other cases, e.g. - // when visiting the Plan page for the first time) - if (this.wereTabsClickedRecently(this.LEARNERS_TABS_ID)) { + // when visiting the page for the first time) + if (this.wereTabsClickedRecently(this.REPORTS_LEARNERS_TABS_ID)) { this.$nextTick(() => { this.$refs.tabList.focusActiveTab(); }); @@ -168,7 +168,7 @@ }, reportLearners: { message: 'Report learners', - context: 'Labels the Reports > Learners tab for screen reander users', + context: 'Labels the Reports > Learners tab for screen reader users', }, }, }; diff --git a/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerReportPage.vue b/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerReportPage.vue index ca4ed1d3299..6f4f2a9259b 100644 --- a/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerReportPage.vue +++ b/kolibri/plugins/coach/assets/src/views/reports/ReportsLearnerReportPage.vue @@ -8,10 +8,10 @@ - + @@ -102,7 +102,7 @@ import commonCoach from '../common'; import CoachAppBarPage from '../CoachAppBarPage'; import { PageNames } from '../../constants'; - import { LEARNERS_TABS_ID, LearnersTabs } from '../../constants/tabsConstants'; + import { REPORTS_LEARNERS_TABS_ID, ReportsLearnersTabs } from '../../constants/tabsConstants'; import ReportsLearnerHeader from './ReportsLearnerHeader'; import ReportsControls from './ReportsControls'; @@ -116,8 +116,8 @@ mixins: [commonCoach, commonCoreStrings], data() { return { - LEARNERS_TABS_ID, - LearnersTabs, + REPORTS_LEARNERS_TABS_ID, + ReportsLearnersTabs, }; }, computed: {