Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jul 29, 2025

Backport of PR #7513

mejo- added 2 commits July 29, 2025 13:36
No need to iterate over all nodes of the document each time.

Upstream PR: amirhhashemi/tiptap-text-direction#23

Signed-off-by: Jonas <jonas@freesources.org>
The upstream Tiptap Keymap extension that we don't use (anymore) also
provides this.

It allows to undo the automatic changes from input rules directly after
they got applied by hitting <Backspace>.

Unfortunately this doesn't work for nodes that get changed by the
TextDirection extension due to a bug, but there's still plenty of input
rules where it works, so let's bring it in.

Signed-off-by: Jonas <jonas@freesources.org>
@backportbot backportbot bot requested a review from max-nextcloud as a code owner July 29, 2025 13:36
@backportbot backportbot bot requested a review from max-nextcloud July 29, 2025 13:36
@backportbot backportbot bot requested a review from mejo- as a code owner July 29, 2025 13:36
@backportbot backportbot bot added bug Something isn't working 3. to review labels Jul 29, 2025
@backportbot backportbot bot added this to the Nextcloud 31.0.8 milestone Jul 29, 2025
@codecov
Copy link

codecov bot commented Jul 29, 2025

Codecov Report

❌ Patch coverage is 69.69697% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.44%. Comparing base (289e7d6) to head (0c856f9).
⚠️ Report is 4 commits behind head on stable31.

Files with missing lines Patch % Lines
src/nodes/Callout.vue 0.00% 4 Missing ⚠️
src/nodes/Table/TableCellView.vue 0.00% 3 Missing ⚠️
src/nodes/Table/TableHeaderView.vue 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           stable31    #7515      +/-   ##
============================================
+ Coverage     51.53%   58.44%   +6.91%     
============================================
  Files           292      291       -1     
  Lines         41433    36559    -4874     
  Branches        871      870       -1     
============================================
+ Hits          21352    21368      +16     
+ Misses        19960    15071    -4889     
+ Partials        121      120       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Fixes styling for details and callouts with RTL script.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the backport/7513/stable31 branch from 1d17f24 to 0c856f9 Compare July 29, 2025 15:11
@mejo-
Copy link
Member

mejo- commented Jul 29, 2025

/backport to stable30

@mejo- mejo- merged commit 2e14db6 into stable31 Jul 29, 2025
64 checks passed
@mejo- mejo- deleted the backport/7513/stable31 branch July 29, 2025 15:42
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