Skip to content

[stable35] fix: comments follow-ups - #9289

Merged
mejo- merged 9 commits into
stable35from
backport/9252/stable35
Sep 28, 2026
Merged

mejo- merged 9 commits into
stable35from
backport/9252/stable35

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 28, 2026

Copy link
Copy Markdown

Backport of PR #9252

Signed-off-by: Jonas <jonas@freesources.org>
Insert comment and then escape now removes the empty comment reference.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
The cleanup plugins for comments and footnotes appended transactions
marked as `addToHistory: alse`. ProseeMirror applies the user's
transaction and the appended ones as one batch and y-prosemirror trakes
the `addToHistory` flag of the last transaction in that batch. This led
to broken undo history.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5.1
* Header: "< Comment 3 of 4 >" instead of "Comments ^ 3 / 4 v"
* Allow to navigate comments with keyboard shortcuts

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5.1
Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-opus-5.5
Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5.1
@backportbot backportbot Bot added bug Something isn't working 3. to review labels Sep 28, 2026
@backportbot backportbot Bot added this to the Nextcloud 35.0.2 milestone Sep 28, 2026
@mejo-
mejo- merged commit 18a6740 into stable35 Sep 28, 2026
75 of 77 checks passed
@mejo-
mejo- deleted the backport/9252/stable35 branch September 28, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant