Commit 26eaa769 authored by Paul Slaughter's avatar Paul Slaughter

Merge branch '197897-enable-mr-single_mr_diff_view-feature-flag-by-default-2' into 'master'

Enable MR :single_mr_diff_view feature flag by default

See merge request gitlab-org/gitlab!24821
parents 5df1c407 ab449502
...@@ -21,7 +21,7 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo ...@@ -21,7 +21,7 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
before_action only: [:show] do before_action only: [:show] do
push_frontend_feature_flag(:diffs_batch_load, @project, default_enabled: true) push_frontend_feature_flag(:diffs_batch_load, @project, default_enabled: true)
push_frontend_feature_flag(:deploy_from_footer, @project, default_enabled: true) push_frontend_feature_flag(:deploy_from_footer, @project, default_enabled: true)
push_frontend_feature_flag(:single_mr_diff_view, @project) push_frontend_feature_flag(:single_mr_diff_view, @project, default_enabled: true)
push_frontend_feature_flag(:suggest_pipeline) if experiment_enabled?(:suggest_pipeline) push_frontend_feature_flag(:suggest_pipeline) if experiment_enabled?(:suggest_pipeline)
end end
......
...@@ -67,14 +67,14 @@ class DiffFileEntity < DiffFileBaseEntity ...@@ -67,14 +67,14 @@ class DiffFileEntity < DiffFileBaseEntity
private private
def parallel_diff_view?(options, diff_file) def parallel_diff_view?(options, diff_file)
return true unless Feature.enabled?(:single_mr_diff_view, diff_file.repository.project) return true unless Feature.enabled?(:single_mr_diff_view, diff_file.repository.project, default_enabled: true)
# If we're not rendering inline, we must be rendering parallel # If we're not rendering inline, we must be rendering parallel
!inline_diff_view?(options, diff_file) !inline_diff_view?(options, diff_file)
end end
def inline_diff_view?(options, diff_file) def inline_diff_view?(options, diff_file)
return true unless Feature.enabled?(:single_mr_diff_view, diff_file.repository.project) return true unless Feature.enabled?(:single_mr_diff_view, diff_file.repository.project, default_enabled: true)
# If nothing is present, inline will be the default. # If nothing is present, inline will be the default.
options.fetch(:diff_view, :inline).to_sym == :inline options.fetch(:diff_view, :inline).to_sym == :inline
......
---
title: Diffs load each view style separately, on demand
merge_request: 24821
author:
type: performance
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