I like the GitLab flow, and I absolutely love the "Create merge request" button in the issue detail. I miss it dearly whenever I'm working on GitHub, and `hub pull-request` isn't quite the same.
I don't like the code review functions: for example when I'm in the changes tab, where I can both resolve discussions and double-check the code, cycling through unresolved discussions breaks every time and I have to go back to the discussions tab where to check the code I have to open the linked diff in another tab.
I don't like the settings UI, with all those "expand" buttons so far on the right side. I mean, there's even one of those buttons in places where you have only one fieldset worth of settings.
Thanks for the feedback. I just spun up a test MR (https://gitlab.com/gitlab-com/www-gitlab-com/merge_requests/...) to understand if I could quickly see what was bothering you on cycling through unresolved discussions but it seems to be working ok. Would it be possible for you to elaborate a bit more or link me to a MR where you can replicate the breaking functionality?
I did notice there's a bug where the hover text doesn't go away after clicking on the "Jump to first unresolved discussion" button and I've created an issue to fix that (https://gitlab.com/gitlab-org/gitlab-ee/issues/15462.)
My biggest issue with gitlab code review is that it's unusable for large changes and large files. On large changes it sometimes won't even show all the changes.
Also the auto-collapse size is far too small, and the fact you can't even see the diff for some large changes in infuriating. It makes using it for code review a struggle. My company is trying to move us all to gitlab-ee, but the lack of features and polish to the code review tool is preventing some teams from making the jump.
I feel like the code review portions are more like a half-circle, and not a heart. There are so many deficiencies that make it unusable for large projects.
If I were gitlab I would look at gerrit for inspiration on what you should use for code review. It's ugly, but it's very functional and performs quite well on meager hardware.
I know there are feature requests to make code review more usable but I can't understand how you could dogfood that code review tool and not go insane.
> it's unusable for large changes and large files. On large changes it sometimes won't even show all the changes.
Improving the performance and scalability of merge requests is the priority for the Source Code team right now, and we are starting with progressively loading the diffs over coming releases. See https://gitlab.com/groups/gitlab-org/-/epics/1816. Currently we load them in a single request which is a serious bottleneck.
In parallel we are exploring a range of UX changes to streamline and improve the code review experience. If you have any specific ideas please let us know, or create an issue!
I like the GitLab flow, and I absolutely love the "Create merge request" button in the issue detail. I miss it dearly whenever I'm working on GitHub, and `hub pull-request` isn't quite the same.
I don't like the code review functions: for example when I'm in the changes tab, where I can both resolve discussions and double-check the code, cycling through unresolved discussions breaks every time and I have to go back to the discussions tab where to check the code I have to open the linked diff in another tab.
I don't like the settings UI, with all those "expand" buttons so far on the right side. I mean, there's even one of those buttons in places where you have only one fieldset worth of settings.