Start a discussion from a non-discussion comment

Applies to

  • Issues
  • Merge requests main discussion area only. (Non-diff discussions.)
  • Epics

Out of scope

  • Merge request diff discussions are out of scope.
  • Commits and snippets are out of scope.

Description

  • Take an existing comment, and turn it into a discussion (by replying to a comment).
  • Hitting the reply button will open up the comment input field and focus the cursor
  • When you first load the page, all discussions are expanded by default.
  • Linked comments will highlight just as they do today

Reply button

  • The reply button only appears if you permissions to reply to an existing discussion or start a discussion from a standalone comment.
  • The reply button does not appear on diff discussions of merge requests, since that is out of scope for this issue.
  • For existing discussions (whether one-comment discussions or multi-comment discussions):
    • The reply button exists on the root comment. If you click it, the reply area comes into focused and you can type in it.
    • The reply button exists on all reply comments. If you click on any one of them, the same reply area comes into focus and you can type in it.
  • For standalone comments:
    • The reply button exists. If you can click on it, the reply area appears and into focus and you can type in it.
  • The mockup below is for merge requests, but it is the same for issues and epics, except with the "resolve" UI.
  • Comment icon: https://gitlab-org.gitlab.io/gitlab-svgs/?q=~comment

[Spec preview] (https://gitlab-org.gitlab.io/gitlab-design/hosted/hazel/ce%2329294-add-container-around-comments-in-mr-discussion-spec-previews/#artboard1) (Please note that these specs are from the original discussion redesign, so things might not match completely. The placement and size of the icon, however, is correct)

Screen_Shot_2019-01-14_at_9.56.08_AM

Todos

  • Initial implementation (!24480 (merged))
  • Properly focusing the input field (!24867 (merged))
  • Fix polling (!25084 (merged))
  • Fix resolvable status (!24950 (merged))
  • Properly revert state when clicking cancel (!25339 (merged))
  • Enable feature flag on GitLab.com
  • Test feature on GitLab.com
  • Remove feature flag, add changelog and docs
Edited Feb 18, 2019 by Constance Okoghenun
Assignee Loading
Time tracking Loading