Skip to content

Change more specs to run with the new sidebar

Lukas Eipert requested to merge leipert-fix-super-sidebar-nav-specs into master

What does this MR do and why?

This is all part of the bigger effort in #420119 (closed) (and a little bit of the related #420121 (closed)) in order to be able to remove the super_sidebar_logged_out feature flag. See also: #419936 (closed)

Fix user snippet and projects specs with new sidebar

Only needed to change the sidebar selector to fix these.

Fix What's New specs with super sidebar

Needed some minor refactors of which buttons to push

Fix Help Dropdown specs with super sidebar

This needed a minor refactor because the DOM structure is slightly different: Instead of a badge with a link to a help page, it is a link to the help page which contains the badge.

Fixes #369850 (closed)

Screenshots or screen recordings

N/A

How to set up and validate locally

Execute the specs 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

Loading