Skip to content

Dry up whether super sidebar should be shown

Joseph Wambua requested to merge jj_391875_dry_show_super_sidebar into master

What does this MR do and why?

Dry up whether super sidebar should be shown

This moves a few instances of current_user.use_new_navigation to use NavHelper.show_super_sidebar? instead. This has the benefit, that we can now change the logic in a single place (e.g. once the feature flag becomes the default).

Furthermore we are able to remove the super_sidebar_supported? private method and it's use, because all sidebars support the super sidebar since: !115162 (merged) is merged.

Fixes: Dry up show_super_sidebar? and gon.use_new_navi... (#391875 - closed) • Lukas Eipert • 16.0

Screenshots or screen recordings

N/A

How to set up and validate locally

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Lukas Eipert

Merge request reports