Skip to content

Hide job sidebar on mobile

Filipa Lacerda requested to merge 52608-sidebar into master

What does this MR do?

Hides job sidebar on mobile:

Screen_Shot_2018-10-12_at_09.36.30

With the refactor, I removed some of the js- classes. I did not take in consideration that https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22116 would not be merged for 11.4, which broke the sidebar behavior.

This MR adds the js- classes back and adds specs to make sure this won't happen again.

What are the relevant issue numbers?

Closes #52608 (closed)

Does this MR meet the acceptance criteria?

Edited by Filipa Lacerda

Merge request reports