Skip to content

Force two up view

Clement Ho requested to merge force-two-up-view into master

What does this MR do?

Forces the two up view to load by default when an image diff contains an updated image

Are there points in the code the reviewer needs to double check?

Shouldn't be

Why was this MR needed?

Fix UI inconsistency and reduce the number of scenarios for image diff comments (https://gitlab.com/gitlab-org/gitlab-ce/issues/35873)

Screenshots (if relevant)

Before (On page load) After (On page load)
Screen_Shot_2017-09-20_at_4.34.59_PM Screen_Shot_2017-09-20_at_4.34.01_PM

Does this MR meet the acceptance criteria?

  • Changelog entry added, if necessary
  • Review
    • Has been reviewed by UX
    • Has been reviewed by Frontend

What are the relevant issue numbers?

Close https://gitlab.com/gitlab-org/gitlab-ce/issues/32613

Edited by Filipa Lacerda

Merge request reports