Details
Sub-task
Description
Get rid of user setting for course display
There is no longer need in per-user setting to show only one section in the course (it confused people anyway).
Remove the control
delete function course_get_display, course_set_display and their usage,
DROP TABLE mdl_course_display
Issue Links
- has been marked as being related by
-
MDL-34829
We should allow 'single section links' to work even if not in single section mode
-
- Closed
-
Activity
Hi Petr,
There are some tests in phpunit_test.php relating to course_display. They are not testing the functionality of course_display, so I was just wondering if they could be replaced by any old table does course_display have a distinctive feature?
feel free to abuse any other empty table
Empty table? Such as?
Petr, I went for user_preferences:
https://github.com/danpoltawski/moodle/commit/67c713ff0f2bebde927af7d5ddc2fdac6ddd602d
yes, it is ok
This has been integrated as part of the pull in MDL-32508
Uh, not sure if anyone else has twigged to this, but I know a whole heap of clients who have been using this method to fight the 'scroll of death' for many years, and disappearing this feature will mean that the only way to display only one topic will be to use the single topic display format, right? What happens to people who have built custom navigation into Topic 1 of their course, or a block, and which is a pivotal part of their learning design?
I hope I'm proved wrong on this, but as someone who has spent a few years working with end users who want as many ways to manage the SoD as possible, this seems like a major mistake in usability to me.
Thoughts/flames welcome.
Hi Mark,
The 'What happens to people who have built custom navigation into Topic 1 of their course, or a block, and which is a pivotal part of their learning design?' was an oversight, and should be fixed in: MDL-34829
This branch removes this setting and gets the topics/weeks to work without this 'sticky setting'.
The navigation link to section was removed and navigation links to sections only display when we are in 'display section at a time' mode.