Skip to content

Conversation

@sundasnoreen12
Copy link
Contributor

@sundasnoreen12 sundasnoreen12 commented Jun 11, 2025

INF-1926

Description
Stacked right sidebar not working correctly with left nav sidebar

Solution
The issue has been resolved by closing the left panel when the unit changes.

The change is related to hiding the course outline sidebar whenever the unit or topic changes. As a result, when navigating to a new unit or topic, the course outline sidebar will automatically close, and only the right panel will remain visible.

@codecov
Copy link

codecov bot commented Jun 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.45%. Comparing base (ab3f5fd) to head (4849d48).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1732   +/-   ##
=======================================
  Coverage   90.44%   90.45%           
=======================================
  Files         344      344           
  Lines        5801     5804    +3     
  Branches     1357     1395   +38     
=======================================
+ Hits         5247     5250    +3     
+ Misses        537      535    -2     
- Partials       17       19    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@awais-ansari
Copy link
Contributor

@nsprenkle Please merge this PR.

@nsprenkle
Copy link
Contributor

nsprenkle commented Jun 12, 2025

@nsprenkle Please merge this PR.

@awais-ansari , I don't have permissions to merge while the codecov check is failing. Please resolve.

@nsprenkle nsprenkle merged commit 7eddc91 into openedx:master Jun 13, 2025
7 checks passed
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.

3 participants