Skip to content

Get rid of 'resolve comment' functionality

Problem to solve

There is a lot of confusion around what the difference between Resolve discussion and Resolve comment is. This is amplified by our UI that relies on icons and tooltips for resolvable comments. It does not help that the two actions are often extremely close to each other and the associated tooltip for a resolvable comment so far said Mark as resolved without mentioning whether it meant the discussion or just the comment.

image

It also creates problems in situations like the following:

  • Person A resolves a comment
  • Person B resolves the discussion
  • Person A unresolves the discussion

In the scenario above, all comments are unresolved again, even though Person A had already resolved the individual comment.

There also is a lot of functionality around the discussions, e.g. unresolved discussions blocking merge requests or the possibility to jump to unresolved discussions. None of this exists for comments.

Proposal

Looking at how often this feature is being used, we should consider getting rid of resolvable comments altogether, so we can avoid this confusion. For every use case I heard so far, there are workarounds such as adding a checkmark icon, which do not have the same problems as our current UI solution and do not make our interface even more complex.

@jramsay You might have heard of good use cases, certainly have a better understanding of how often it's being used and how valuable it is to our users, would love to hear your opinion.

What does success look like, and how can we measure that?

No pushback from our community.

Edited by Marcel van Remmerden