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

[stable28] Fix/toc without transaction #6124

Merged
merged 10 commits into from
Jul 29, 2024
Merged

[stable28] Fix/toc without transaction #6124

merged 10 commits into from
Jul 29, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jul 25, 2024

@backportbot backportbot bot requested a review from mejo- July 25, 2024 17:43
@backportbot backportbot bot added bug Something isn't working 3. to review labels Jul 25, 2024
@backportbot backportbot bot added this to the Nextcloud 28.0.9 milestone Jul 25, 2024
@mejo- mejo- force-pushed the backport/6116/stable28 branch 2 times, most recently from b2e4606 to a4f5ffb Compare July 25, 2024 17:49
@mejo- mejo- marked this pull request as ready for review July 25, 2024 17:51
@mejo- mejo- changed the title [stable28] [stable29] Fix/toc without transaction [stable28] Fix/toc without transaction Jul 26, 2024
max-nextcloud and others added 10 commits July 29, 2024 09:54
Signed-off-by: Max <max@nextcloud.com>
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>
Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the backport/6116/stable28 branch from a4f5ffb to 8a6768a Compare July 29, 2024 07:54
@mejo- mejo- merged commit 6f4f629 into stable28 Jul 29, 2024
58 checks passed
@mejo- mejo- deleted the backport/6116/stable28 branch July 29, 2024 08:07
@Altahrim Altahrim mentioned this pull request Aug 8, 2024
3 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