Impossible to remove commit_id from url params in MR using the UI
Summary
When reviewing a merge request, I may want to see a particular commit, but when I go back to the MR overview, I want to see all MR diffs again.
Steps to reproduce
- Go to a merge request which contains more than one commit
- Click on the "Commits" tab and select a commit - this brings us to the "Changes" tab and adds the
commit_id
url param, so we only see the changes related to this commit - Click on the "Overview" tab -> at this point, I expect to see "MR-level" changes again
- Go again to "Changes" tab -> because the
commit_id
is still present, I still see the previously selected commit's changes
Expected behaviour
When leaving the "Changes" section after seeing a particular commit's diff, commit_id
should be removed from the params
Further comments
It may be argued that keeping the commit_id
is a desired behavior, which I find not practical, but then, how does one go back to seeing MR-level changes without removing commit_id
manually from the url?
Edited by Enzo Parodi