Skip to content

[BUG FIX] [MER-5887] Learning Objectives Authoring Elements Corrections - #6792

Open
gastonabella wants to merge 12 commits into
masterfrom
MER-5887-LOs-authoring-corrections
Open

[BUG FIX] [MER-5887] Learning Objectives Authoring Elements Corrections#6792
gastonabella wants to merge 12 commits into
masterfrom
MER-5887-LOs-authoring-corrections

Conversation

@gastonabella

@gastonabella gastonabella commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR completes the authoring-side corrections for the Learning Objectives Introduction/Summary feature originally introduced in #6745.

It focuses on the authoring requirements from:

  • MER-5802: authoring insert menu support for the Learning Objectives element
  • MER-5803: authoring support for Learning Objective Introduction
  • MER-5804: authoring support for Learning Objective Summary

Changes

  • Updated the Insert menu Objectives entry to match the intended icon, placement, hover/focus behavior, and root-only insertion behavior.
  • Refined the authoring layout for Learning Objective Introduction and Summary, including descriptions, mode selector placement, panel structure, spacing, warning placement, and proficiency accordion.
  • Updated LO cards to match the expected authoring structure, including LO X labels, parent/sub-objective grouping, recommendation selectors, chips, disabled add actions, and tooltip behavior.
  • Added non-destructive remove/restore behavior for objectives, preserving recommendation config while toggling enabled.
  • Improved the empty-LO warning banner, including warning icon, dismiss action, colors, and no-false-warning behavior while LOs are unresolved.
  • Added the shared trash icon treatment for authoring delete buttons and LO remove actions.
  • Added authoring dark mode color fixes for LO surfaces, cards, warnings, chips, controls, and proficiency blocks.
  • Added focused Jest coverage for insert behavior, mode switching, hierarchy rendering, remove/restore preservation, recommendation actions, empty warning behavior, and unresolved LO state.

Notes

The student/delivery corrections from MER-5867 are intentionally left for a follow-up PR covering MER-5807 and MER-5808.

Screen.Recording.2026-08-21.at.3.56.24.PM.mov

@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor
Warnings
⚠️ PR is large (2526 LOC changed). Consider splitting.

Risk score: 16 → risk/high

Generated by 🚫 dangerJS against 48065ce

@gastonabella gastonabella changed the title Mer 5887 l os authoring corrections [BUG FIX] [MER-5887] Learning Objectives Authoring Elements Corrections Aug 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

AI Review — ui

No issues found

@github-actions

Copy link
Copy Markdown
Contributor

AI Review — typescript

No issues found

@github-actions

Copy link
Copy Markdown
Contributor

AI Review — security

No issues found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant