Skip to content

Draft: Added new lines property to diff files

Phil Hughes requested to merge ph/236143/addedNewLinesPropToDiffFiles into master

What does this MR do?

The new lines property is the same data structure for both inline and parallel views, they just get rendered with different components.

Whilst this merge request allows for the different views to be rendered from the same data structure, it does not however allow for full feature parity. Certain parts do not work in this, which is ok as it is all behind a feature flag.

Screenshots

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

Merge request reports