This issue is part of the course hierarch project.

Issue goals

Once activities can act as subsections (mod_subsection), the move activity modal should also display that kind of activity.

This mean:

• Any activity with a delegated section integration must be displayed as a tree node with all subactivities listed under it.
• Clicking the mod_subsection will move the activity on top of the subsection. However, this has a limitation because there is no way to way to indicate you want the activity just under the subsection. This will require a UX review (there is a larger explanation on that in the next section).
• Validate the keyboard navigation still works as expected.

Ux review required

The current modal has the following behaviour:

• Clicking on a section moves the element on top of that section
• Clicking on an activity moves the element under the activity

This approach was the best for a single-level hierarchy because the user can move the activity anywhere in the course without limitation. However, the mod_subsection will act as both a section and an activity, creating a problem.

If mod_subsection acts only as a section, moving an element just below the subsection is impossible. Similarly, if it acts as an activity, moving an element on top of the subsection is impossible.

There are a couple of approaches I can think of:

1. Assume that limitation and implement only the most useful one. This will force some movements to use twice the move action to get some specific positions.
2. Add a new link in the modal to cover those particular cases. For example, adding a "move under" next to the mod_subsection name or a "move on top" just under the mod_subsection name. In the end, it implies adding an extra "move here" link for the locations that are not selectable.

Future issues

For accessibility reasons, activities must be moved using a click UI rather than drag and drop, so this issue must be resolved before integrating the mod_subsection into the core.