Closed rohitpaulk closed 1 week ago
The changes in the pull request focus on the course.hbs
template, enhancing its structure and conditional rendering for the course page. Key modifications include the introduction of did-insert
and did-update
hooks for managing current step values, adjustments to the sidebar components for desktop and mobile views, and a restructured leaderboard section. The leaderboard buttons are now within a sticky div
, and the CourseLeaderboard
component is nested accordingly. Visibility conditions for certain components have been retained, ensuring existing functionality remains intact.
File | Change Summary |
---|---|
app/templates/course.hbs | - Added did-insert and did-update hooks for current step management.- Retained sidebar components with conditional rendering for desktop and mobile. - Altered layout of the leaderboard section, moving buttons into a sticky div .- Nested CourseLeaderboard within the new div for sticky positioning.- Preserved visibility condition for PrivateLeaderboardFeatureSuggestion . |
In the course of change, we hop and play,
With sticky buttons guiding the way.
Sidebars expand, both near and far,
As we leap through learning, like a shining star.
Each step we take, a new delight,
In our cozy template, everything feels right! πβ¨
ββ1 filesβ Β±0ββββ1 suitesβ Β±0βββ6m 33s :stopwatch: +8s 574 tests Β±0ββ538 :white_check_mark: Β±0ββ36 :zzz: Β±0ββ0 :x: Β±0β 589 runsβ Β±0ββ553 :white_check_mark: Β±0ββ36 :zzz: Β±0ββ0 :x: Β±0β
Results for commit 8b3cf1dd.βΒ± Comparison against base commit 66b211fc.
All modified and coverable lines are covered by tests :white_check_mark:
:white_check_mark: All tests successful. No failed tests found.
:loudspeaker: Thoughts on this report? Let us know!
Checklist:
[percy]
in the message to trigger)Summary by CodeRabbit
New Features
Bug Fixes