On 03/12/13 01:07, alin wrote:
HI guys,
Thank you very much for all the hard work that got into v3, great job. The new
look is quite awesome and i find the overall performance to be excellent, so
much faster than the previous version.
I guess this email can be read as a feature request/ bug fix centered around
the merge request app. which i use mainly for code reviews. I do realize the
merge request tool is still been worked on and more feature will come, just
wanted to give you a short list of my findings/gripes:
1. no way to go back to merge request page from the diff tool. when i get a
merge request i click on 'View All Diffs' and i start adding comments. once i'm
done commenting i would like to go back and change the merge request state. I
couldn't find any way to go back to the merge request page other than go to the
'Merge Requests' list, find my merge request and click on it (or edit the URL
in the browser). Have i missed something?
2. Add a comment on multiple lines. previous version of gitorious used to have
this feature and i was using it all the time.
3. merge request commits list. This is quite confusing, if i click on a commit
id from a merge request commits list, the diff that i see is not actually in
the context of the merge request. meaning that if i add a comment for example,
the comment is 'global', not part of the merge request.
4. navigate from comments to where in the merge request the comment has been
added. when i look at a merge request and i see that somebody has added a
comment, i now see the line on which the comment has been added (awesome). it
will be extremely useful if i could easily click on the comment and have the UI
navigate my to the appropriate place in the diff where the comment has been
added.
5. expand/collapse files in 'view all diffs'. we used to have this feature in
the previous version. very useful on large merge requests
Regards,
Alin
Hi Alin,
Thanks for feedback and for this list. These are the things we all had
in our heads for some time too. Would you mind adding these ideas as a
separate issues here? https://gitorious.org/projects/gitorious/issues/new
Marcin
--
--
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
---
You received this message because you are subscribed to the Google Groups "Gitorious" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
