Skip to content

Work item threads/comments - make add actions appear as inputs

What does this MR do and why?

Closes Work item threads/comments - make add actions a... (#393976 - closed).

Scope of changes :

  • WI thread reply button -> textarea
  • Ability to close/reopen issue from the main/fresh comment
  • Fresh comment always available in edit mode

Screenshots or screen recordings

Fresh comment

Before(Fresh comment) After(Fresh comment)
Screenshot_2023-03-29_at_3.07.52_PM Screenshot_2023-03-29_at_3.08.07_PM

Reply to a thread , default , isEditing false

Before(reply to thread) After(reply to thread)
Screenshot_2023-03-29_at_3.08.20_PM Screenshot_2023-03-29_at_3.08.28_PM

Reply to a thread , triggered click/focus , isEditing true

Before(reply to thread) After(reply to thread)
Screenshot_2023-03-29_at_3.08.49_PM Screenshot_2023-03-29_at_3.08.37_PM

Detailed screenshots

header header
total padding 16px Screenshot_2023-04-04_at_11.47.32_AM
Button label for replies should be "Reply", not "Comment" (new comments, not in thread, remain comment) Screenshot_2023-04-04_at_11.48.15_AM Screenshot_2023-04-04_at_11.48.26_AM
Reply focus ring should not be cut off in any mode Screenshot_2023-04-04_at_11.49.09_AM. Screenshot_2023-04-04_at_11.49.16_AM
Clicking the "Reply" button should automatically focus the editor, currently shows the editor but does not focus Screen_Recording_2023-04-04_at_11.51.30_AM
Distance should be 16px margin between last item and comment form Screenshot_2023-04-04_at_11.54.04_AM Screenshot_2023-04-04_at_11.54.31_AM Screenshot_2023-04-04_at_11.54.42_AM Screenshot_2023-04-04_at_11.54.51_AM

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Open any work item -> task , objective or key result
  2. Try to add a fresh comment / reply to threads and see the visual difference

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Deepika Guliani

Merge request reports

Loading