Skip to content

Commit c45de88

Browse files
author
Filipa Lacerda
committed
Merge branch 'mr-file-tree-inline-fluid-width-fix' into 'master'
Fixed merge request fill tree not respecting fluid width Closes #52916 See merge request gitlab-org/gitlab-ce!22487
2 parents ffc505d + 369bb6f commit c45de88

File tree

3 files changed

+14
-1
lines changed

3 files changed

+14
-1
lines changed

app/assets/javascripts/merge_request_tabs.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,7 @@ export default class MergeRequestTabs {
419419
if (this.diffViewType() === 'parallel' || removeLimited) {
420420
$wrapper.removeClass('container-limited');
421421
} else {
422-
$wrapper.addClass('container-limited');
422+
$wrapper.toggleClass('container-limited', this.fixedLayoutPref);
423423
}
424424
}
425425

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
title: Fixed merge request fill tree toggling not respecting fluid width preference
3+
merge_request:
4+
author:
5+
type: fixed

spec/javascripts/merge_request_tabs_spec.js

+8
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,14 @@ describe('MergeRequestTabs', function() {
224224
expect($('.content-wrapper')).not.toContainElement('.container-limited');
225225
});
226226

227+
it('does not add container-limited when fluid layout is prefered', function() {
228+
$('.content-wrapper .container-fluid').removeClass('container-limited');
229+
230+
this.class.expandViewContainer(false);
231+
232+
expect($('.content-wrapper')).not.toContainElement('.container-limited');
233+
});
234+
227235
it('does remove container-limited from breadcrumbs', function() {
228236
$('.container-limited').addClass('breadcrumbs');
229237
this.class.expandViewContainer();

0 commit comments

Comments
 (0)