Course builder improvements

[brianhogg]

Description

• Show permalink for new lessons and quizzes right after saving
• Quiz set to Published by default
• Create named quiz url using title if none manually entered
• Add lesson description (content)
• Hide the "Add Existing Lesson" modal after selecting a lesson
• Improved section selection, and highlighting of the current section

Fixes #3033, #3056, #3097, #2938, #3030

How has this been tested?

Screenshots

Types of changes

Checklist:

• This PR requires and contains at least one changelog file.
• My code has been tested.
• My code passes all existing automated tests.
• My code follows the LifterLMS Coding & Documentation Standards.

---

Note

Medium Risk
Changes touch both the course builder sync pipeline (JS + PHP) and lesson/quiz persistence, including new logic to conditionally overwrite lesson content. Risk is moderate because regressions could affect saving behavior and sidebar/editor UX across lessons, quizzes, and assignments.

Overview
Improves the Course Builder editing flow by adding in-builder lesson content editing (with a new content_added_in_builder flag) and preventing builder saves from overwriting existing lesson content that was created outside the builder.

Updates the sync/save response handling so newly saved lessons/quizzes/assignments can immediately display refreshed permalinks/slugs/titles without full re-renders, and sets new quizzes to publish by default (with quiz posts created using the provided title to generate a named URL).

Polishes builder UX: closes the sidebar editor when the selected model is trashed, ensures the “Add Existing Lesson” popover reliably closes after selection, and refines section selection/highlighting (auto-select first section on load, broader click target, improved selected styling).

^{Reviewed by Cursor Bugbot for commit 1809a9b. Bugbot is set up for automated code reviews on this repo. Configure here.}

[brianhogg]

@cursor review

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 1809a9b. Configure here.}

[cursor]

Stale event listener from broken onHide cleanup

Medium Severity

The onHide callback calls Backbone.pubSub.off('lesson-search-select', hidePopover), but hidePopover was never directly registered as a listener. The actual listener is the anonymous function passed to Backbone.pubSub.once(...). So the off call is a no-op and never cleans up the once handler. If the user dismisses the popover without selecting a lesson and then reopens it, stale once handlers accumulate. On the next lesson selection, all stale handlers fire, calling pop.hide() on old references and running $('.webui-popover').remove() multiple times, potentially removing unrelated popovers.

Additional Locations (1)

• assets/js/builder/Views/Elements.js#L159-L166

 

^{Reviewed by Cursor Bugbot for commit 1809a9b. Configure here.}

[cursor]

Missing change:name listener in add_existing_quiz

Low Severity

add_existing_quiz registers change:permalink but not change:name on the new model, unlike add_new_quiz which registers both. For cloned quizzes, the server generates a new slug returned as name. In the sync handler, permalink is set before name, so change:permalink triggers render_settings before name is updated — rendering the settings with a stale slug value that never gets corrected.

 

^{Reviewed by Cursor Bugbot for commit 1809a9b. Configure here.}