Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-76188

Course editor: Migrate highlight/unhighlight topics to new reactive actions.

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 4.1
    • 4.2
    • Course
    • 3
    • HQ Sprint 1.2 Moppies

    Description

      Since Moodle 4.0 the course editor frontend part uses two main modules to handle the edition tools:

      • core_course/actions: responsible for handling all non-reactive tools and the legacy formats
      • core_courseformat/courseeditor: the new reactive library that updates the course index and the course content

      Since the reactive implementation starts, the final goal is to replace all legacy actions with new ones as soon as possible. The most complex ones like moving elements, deleting, and creating sections are already migrated and others like indent are just deprecated (or leave it to third-party plugins).

      In this issue the visibility tools will be migrated to the new module:

      • Highlight section

      This feature is used only in the topics format. Since Moodle 3.11 this feature was half implemented in the core and half in the topics format creating some inconsistencies. When this issue is integrated, all the highlight logic will be moved to the topics format plugin. Only the most internal methods will remain in the code implementation. This change has the extra benefit of having a good example of how to extend the current course editor features within a plugin.

      There's a proof of concept of the highlight option migrated entirely to the topics format: https://github.com/ferranrecio/moodle/commit/e15fcf0d85534c1c78c3df70077b9e6e946c1fcf

       

      Attachments

        Issue Links

          Activity

            People

              tusefomal Ferran Recio
              tusefomal Ferran Recio
              Carlos Escobedo Carlos Escobedo
              Paul Holden Paul Holden
              CiBoT CiBoT
              Amaia Anabitarte, Carlos Escobedo, Laurent David, Sabina Abellan, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 2 days, 2 hours
                  2d 2h