1105155 - Indicate that an existing comment has been modified for tracking flags with prefill text

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Jesse Ruderman]

While editing bug 1104985, choosing the tracking flag changed my comment from [X] to [this should block because X]. Which makes me look bad and makes it less likely that [X] or the bug will be fixed.

Changing the meaning of my comment is NEVER OKAY. Please remove this misfeature (which was added in bug 853108).

Specifically, it silently added "[Tracking Requested - why for this release]:" to the top of my comment.

Comment 1

[:glob ✱]

the comments are not "garbled" - the template text is prepended to existing contents.

i don't think that removing this feature is the correct course of action.
we should instead provide feedback to the user that their comment has been updated -- we can change the text of the "comment required" text to "comment updated" in this case.

Comment 2

[:glob ✱]

Attachment: 1105155_1.patch

Comment 3

[David Lawrence [:dkl]]

Comment on attachment 8529086 [details] [diff] [review]
1105155_1.patch

Review of attachment 8529086 [details] [diff] [review]:
-----------------------------------------------------------------

Is good. r=dkl

Comment 4

[:glob ✱]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   9addfeb..1935ede  master -> master