Skip to content
Snippets Groups Projects
Commit 9ebafb47 authored by Jan-Hendrik Willms's avatar Jan-Hendrik Willms Committed by Jan-Hendrik Willms
Browse files

show visibility toggle on content modules administration page only for courses, fixes #4420

Closes #4420

Merge request studip/studip!3244
parent 20bc39a5
No related branches found
No related tags found
No related merge requests found
<div class="content-modules-vue-app" is="ContentModules"></div> <div class="content-modules-vue-app"
is="ContentModules"
range-type="<?= get_class($sem) ?>"
></div>
...@@ -36,6 +36,7 @@ ...@@ -36,6 +36,7 @@
<component :is="displayComponent" <component :is="displayComponent"
:modules="modules" :modules="modules"
:filtercategory="filterCategory" :filtercategory="filterCategory"
:range-type="rangeType"
></component> ></component>
<MountingPortal mount-to="#tool-view-switch .sidebar-widget-content .widget-list" name="sidebar-switch"> <MountingPortal mount-to="#tool-view-switch .sidebar-widget-content .widget-list" name="sidebar-switch">
......
...@@ -61,7 +61,7 @@ ...@@ -61,7 +61,7 @@
href="#" href="#"
class="toggle_visibility" class="toggle_visibility"
role="checkbox" role="checkbox"
v-if="!module.mandatory" v-if="showVisibilityToggle(module)"
:aria-checked="module.visibility !== 'tutor' ? 'true' : 'false'" :aria-checked="module.visibility !== 'tutor' ? 'true' : 'false'"
@click.prevent="toggleModuleVisibility(module)" @click.prevent="toggleModuleVisibility(module)"
> >
......
...@@ -59,7 +59,7 @@ ...@@ -59,7 +59,7 @@
<td class="actions"> <td class="actions">
<a <a
href="#" href="#"
v-if="module.active && !module.mandatory" v-if="showVisibilityToggle(module)"
role="checkbox" role="checkbox"
:aria-checked="module.visibility !== 'tutor' ? 'true' : 'false'" :aria-checked="module.visibility !== 'tutor' ? 'true' : 'false'"
@click.prevent="toggleModuleVisibility(module)" @click.prevent="toggleModuleVisibility(module)"
...@@ -121,7 +121,6 @@ import ContentModulesMixin from '../mixins/ContentModulesMixin.js'; ...@@ -121,7 +121,6 @@ import ContentModulesMixin from '../mixins/ContentModulesMixin.js';
export default { export default {
name: 'contentmodules-edit-table', name: 'contentmodules-edit-table',
mixins: [ContentModulesMixin], mixins: [ContentModulesMixin],
}; };
</script> </script>
......
...@@ -5,6 +5,9 @@ export default { ...@@ -5,6 +5,9 @@ export default {
components: { components: {
draggable, draggable,
}, },
props: {
rangeType: String,
},
data: () => ({ data: () => ({
order: [], order: [],
}), }),
...@@ -132,5 +135,10 @@ export default { ...@@ -132,5 +135,10 @@ export default {
return classes.join(' '); return classes.join(' ');
}, },
showVisibilityToggle(module) {
return this.rangeType !== 'Institute'
&& module.active
&& !module.mandatory;
}
}, },
}; };
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment