- Issue created by @catch
- 🇨🇦Canada ob3ron Canada
Removing "PP-1" as 📌 Update legacy themes Active has landed, and adding as a child issue of 📌 Improve general appearance Active
- 🇨🇦Canada ob3ron Canada
WIP commit - includes a number of navigation improvements:
- adds single-directory components (SDC) for layout and styling.
- displays all but the current lesson as collapsed accordion lists, for compactness and better visual progress display.
- removes the link url from activities that are not available to visit, based on the course-level Free Navigation setting, or if the user is revisiting a completed course.
- improves the course navigation caching/placeholder logic, with appropriate cache lifetimes based on per-user and per-lesson display options.
- adds styling to each activity to indicate current / answered / unanswered activities.
- adds activity scores on course revisit.
- 🇨🇦Canada ob3ron Canada
Will do additional code cleanup and optimization, meanwhile any feedback on the looks and function is welcome.
- 🇬🇧United Kingdom catch
Haven't reviewed the MR or functionality yet, but just changing status to make it clear there's something to look at here.
- 🇨🇦Canada ob3ron Canada
Added additional caching improvements and testing. Should be ready for review.
- First commit to issue fork.
- 🇨🇦Canada ob3ron Canada
Sorry about the accidental test changes, not sure how that happened!
- 🇵🇱Poland Graber
No worries, just merged 📌 Improve / fix course navigation logic Active , let's use training manager API as much as possible here.
- Assigned to ob3ron
- Status changed to Needs review
about 1 month ago 6:27am 3 June 2025 - 🇨🇦Canada ob3ron Canada
Updated BlockBuilder to depend on TrainingManager as the single source of truth for activity access.
Also added a progress bar and a few other tweaks to the navigation block.
- 🇬🇧United Kingdom catch
Added some comments on the MR - I think the locking can be removed and the caching logic potentially simplified a bit.
- 🇵🇱Poland Graber
Cache is enabled.
All this will need extending automated tests coverage but I'll get to that in a separate issue. - 🇵🇱Poland Graber
Merged and created one follow-up to optimize this a bit ( 📌 Course nav - additional caching optimization Active ).
Automatically closed - issue fixed for 2 weeks with no activity.