Add diff_id to diff file links
What does this MR do and why?
This MR:
-
Adds
diff_id
to diff file:- Line links
- Header link
- Refactors the
mapParallel
code so the dependencies are clear - Fixes a bug when no scrolling occurred on file header click (
handleFileNameClick
callback was given a wrong element selector) - Refactors
lineHref
and related code to useURL
class to generate file links. This ensures all the query params are preserved for file links.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
How to set up and validate locally
- Open any merge request with at least 2 versions (pushes)
- Select a version that is not the latest one
- Copy a direct link to one of the diff lines in a file (not the current page URL)
- Open that link in a new tab
- The version of the MR should be correct
Edited by Stanislav Lashmanov