Skip to content

Preserve the wording each comment refers to - #2456

Merged
tf merged 1 commit into
codevise:masterfrom
tf:comment-quote
Jul 30, 2026
Merged

Preserve the wording each comment refers to#2456
tf merged 1 commit into
codevise:masterfrom
tf:comment-quote

Conversation

@tf

@tf tf commented Jul 30, 2026

Copy link
Copy Markdown
Member

Editing text after a comment was written left the conversation without a point of reference: nothing showed what the wording had been when someone objected to it. Longer threads made this worse, since later replies can refer to yet another intermediate version.

Each comment now keeps the wording it responded to and shows it once the text has changed, so threads stay readable after the fact. Comments on text that still reads the same show nothing, leaving the common case as compact as before.

Available for text blocks, where a comment covers a selected range, and for headings, where it covers the whole element. Comments written earlier have no recorded wording and show none.

REDMINE-21261

Editing text after a comment was written left the conversation
without a point of reference: nothing showed what the wording had
been when someone objected to it. Longer threads made this worse,
since later replies can refer to yet another intermediate version.

Each comment now keeps the wording it responded to and shows it
once the text has changed, so threads stay readable after the fact.
Comments on text that still reads the same show nothing, leaving
the common case as compact as before.

Available for text blocks, where a comment covers a selected range,
and for headings, where it covers the whole element. Comments
written earlier have no recorded wording and show none.

REDMINE-21261
@tf tf added this to the v17.2 milestone Jul 30, 2026
@tf
tf merged commit 09c1916 into codevise:master Jul 30, 2026
18 checks passed
@tf
tf deleted the comment-quote branch July 30, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant