Skip to content

feat: Update Xpert Summary frontend widget and location#402

Open
alenkadev wants to merge 1 commit into
release-ulmofrom
alenka/Update-Xpert-Summary-frontend-widget-and-location
Open

feat: Update Xpert Summary frontend widget and location#402
alenkadev wants to merge 1 commit into
release-ulmofrom
alenka/Update-Xpert-Summary-frontend-widget-and-location

Conversation

@alenkadev

Copy link
Copy Markdown

Summary

Moves the Xpert Unit Summary button to render above a unit's content instead of below it.

XBlockAsides render after their block's content by default (xblock.runtime.Runtime.layout_asides), which put the summary button at the bottom of every unit. This adds an LMS-specific override, lms_layout_asides, that renders summaryhook_aside before the block's content instead — every other aside keeps the default after-block placement, so this only affects the Xpert Summary widget.

Changes

  • lms/djangoapps/lms_xblock/runtime.py: new lms_layout_asides() function
  • lms/djangoapps/courseware/block_render.py: registers it via runtime.layout_asides_override

jira-link- LP-871

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants