Skip to content

Replace SidebarHeightManager by CSS rules

Winnie Hellmann requested to merge winh-newnav-issuable-sidebar into master

What does this MR do?

Remove SidebarHeightManager and use CSS rules instead.

Why was this MR needed?

Too many factors influenced the sidebar height leading to several regressions.

Screenshots

new-bulk-issue-sidebar

new-issue-board-sidebar

new-issue-sidebar

new-job-sidebar

What are the relevant issue numbers?

fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/36447, fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/36635, fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/36855

Edited by Winnie Hellmann

Merge request reports