Commit fc247b15 authored by Filipa Lacerda's avatar Filipa Lacerda

Merge branch 'remove-expand-diff-feature-flag' into 'master'

Remove expand diff to full file feature flag

See merge request gitlab-org/gitlab-ce!26614
parents 0f058ff3 3b158409
...@@ -128,9 +128,6 @@ export default { ...@@ -128,9 +128,6 @@ export default {
isModeChanged() { isModeChanged() {
return this.diffFile.viewer.name === diffViewerModes.mode_changed; return this.diffFile.viewer.name === diffViewerModes.mode_changed;
}, },
showExpandDiffToFullFileEnabled() {
return gon.features.expandDiffFullFile && !this.diffFile.is_fully_expanded;
},
}, },
mounted() { mounted() {
polyfillSticky(this.$refs.header); polyfillSticky(this.$refs.header);
...@@ -258,7 +255,7 @@ export default { ...@@ -258,7 +255,7 @@ export default {
<icon name="external-link" /> <icon name="external-link" />
</gl-button> </gl-button>
<gl-button <gl-button
v-if="showExpandDiffToFullFileEnabled" v-if="!diffFile.is_fully_expanded"
class="expand-file js-expand-file" class="expand-file js-expand-file"
@click="toggleFullDiff(diffFile.file_path)" @click="toggleFullDiff(diffFile.file_path)"
> >
......
...@@ -16,10 +16,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo ...@@ -16,10 +16,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
before_action :authenticate_user!, only: [:assign_related_issues] before_action :authenticate_user!, only: [:assign_related_issues]
before_action :check_user_can_push_to_source_branch!, only: [:rebase] before_action :check_user_can_push_to_source_branch!, only: [:rebase]
before_action only: [:show] do
push_frontend_feature_flag(:expand_diff_full_file)
end
def index def index
@merge_requests = @issuables @merge_requests = @issuables
......
...@@ -57,7 +57,7 @@ class DiffFileEntity < DiffFileBaseEntity ...@@ -57,7 +57,7 @@ class DiffFileEntity < DiffFileBaseEntity
diff_file.diff_lines_for_serializer diff_file.diff_lines_for_serializer
end end
expose :is_fully_expanded, if: -> (diff_file, _) { Feature.enabled?(:expand_diff_full_file, default_enabled: true) && diff_file.text? } do |diff_file| expose :is_fully_expanded, if: -> (diff_file, _) { diff_file.text? } do |diff_file|
diff_file.fully_expanded? diff_file.fully_expanded?
end end
......
...@@ -23,9 +23,6 @@ describe('diff_file_header', () => { ...@@ -23,9 +23,6 @@ describe('diff_file_header', () => {
}); });
beforeEach(() => { beforeEach(() => {
gon.features = {
expandDiffFullFile: true,
};
const diffFile = diffDiscussionMock.diff_file; const diffFile = diffDiscussionMock.diff_file;
diffFile.added_lines = 2; diffFile.added_lines = 2;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment