[go: up one dir, main page]

Skip to content

Fix parallel diff expansion line numbers

What does this MR do?

For the diff expansion part of #205938 (closed), which only happens on collapsed diffs in the middle of the file in parallel view.

The diff expansion cell component expects the inline lines to be present regardless of which view type is being used. This MR updates the logic to check the appropriate diff type.

Context

This regression was originally caused by !19930 (merged) (merged over 1 month ago), but wasn't exposed until testing the feature across all of GitLab (#197897 (comment 288703756)).

Screenshots

Before After
before-compressed after-compressed

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • ~~Label as security and @ mention @gitlab-com/gl-security/appsec~~
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading