[task_dispensers] grouped actions on tasks #965
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR follows #964 and proposes a first implementation of grouped actions on tasks in TOC-based dispensers. This had become a must-have in order to separate course from task sets and move the course data in database only, making the task dispenser settings not modifiable from the files.
This implementation is HTML/JS based only, quite heavy and probably not the cleanest possible (it produces a lot of js code). However, I think it's worth to evaluate the user experience first before refactoring and making this definitive.
The implementation is based on the following requirements:
and resolves this by:
This whole UI, while still not being the optimal solution, is slowly becoming an implementation nightmare. It's probably the first signs that jQuery is not sufficient anymore as a Javascript frontend framework.
Another issue: this implementation is still not smartphone friendly. However, the main issue is the drag and drop that prevents appropriate usage of the rest of the UI. I think that either toggling on and off the drag and drop, or completely removing it, might be a first solution. But this would be for a future PR...