You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With AffixedSidebar, the side nav uses position: fixed for side nav's ul to make it fixed on viewport. However, in #3876, the margin-top of ul is moved to the AffixedSidebar's parent element Page-sidebar.
From the top value used below, we can see it only includes the height of header, so the side nav is at a taller position than 1.x.
Current Behavior
With AffixedSidebar, the side nav uses
position: fixed
for side nav'sul
to make it fixed on viewport. However, in #3876, themargin-top
oful
is moved to the AffixedSidebar's parent elementPage-sidebar
.From the
top
value used below, we can see it only includes the height of header, so the side nav is at a taller position than 1.x.framework/framework/core/less/common/sideNav.less
Line 66 in 4feb4a3
Steps to Reproduce
Expected Behavior
The missing offset provided by
margin-top
should be still there.Screenshots
Environment
Output of
php flarum info
No response
Possible Solution
Backport 11fc74d and ea21ef4, which uses
position: sticky
and adds the missing offset.Additional Context
No response
The text was updated successfully, but these errors were encountered: