Skip to content
GitLab Next
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • GitLab GitLab
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 43,823
    • Issues 43,823
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1,410
    • Merge requests 1,410
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.org
  • GitLabGitLab
  • Issues
  • #28750
Closed
Open
Created May 15, 2019 by Marcel van Remmerden@mvanremmerdenDeveloper

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.

Note: the top comment resolve checkmark now needs to resolve the whole thread and not just the individual root note. See: #28750 (comment 342982972)

@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.

Future work

  1. Clean up backend tech debt of parts that are no longer used (since we're removing the ability to resolve per comment). The API could be part of this.
Edited Oct 21, 2020 by Pedro Moreira da Silva
Assignee
Assign to
Time tracking