Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable29] Fix/toc without transaction #6116

Merged
merged 9 commits into from
Jul 25, 2024
Merged

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jul 25, 2024

Signed-off-by: Max <max@nextcloud.com>
@backportbot backportbot bot requested review from mejo- and max-nextcloud July 25, 2024 12:33
@backportbot backportbot bot added bug Something isn't working 3. to review labels Jul 25, 2024
@backportbot backportbot bot added this to the Nextcloud 29.0.5 milestone Jul 25, 2024
max-nextcloud and others added 7 commits July 25, 2024 16:02
This avoids transactions that actually change the document state.

Fixes #5861.

Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: Max <max@nextcloud.com>
This will trigger an update event on the editor
which in turn will trigger all update handlers.

Avoid this when not necessary.

Signed-off-by: Max <max@nextcloud.com>
This allows typing even at the beginning of the heading
as the cursor now is between the anchor and the first letter.

Signed-off-by: Max <max@nextcloud.com>
The toc does not trigger a transaction anymore
and therefore there is no push to wait for either.

Also use cypress aliases and avoid deep nesting
of cypress calls when possible.

Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: Max <max@nextcloud.com>
When the whole document gets replaced by the same content,
`DecorationSet.map()` returns an empty decorationSet. So only use it for
updates where no decorations get removed.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the backport/5893/stable29 branch from 301341d to dc2ecd2 Compare July 25, 2024 14:07
@mejo- mejo- marked this pull request as ready for review July 25, 2024 14:07
@mejo-
Copy link
Member

mejo- commented Jul 25, 2024

/backport to stable28

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- merged commit 9fc1852 into stable29 Jul 25, 2024
57 of 59 checks passed
@mejo- mejo- deleted the backport/5893/stable29 branch July 25, 2024 17:42
@Altahrim Altahrim mentioned this pull request Aug 8, 2024
9 tasks
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.

2 participants