Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jul 29, 2025

Backport of #7515

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

No need to iterate over all nodes of the document each time.

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

Signed-off-by: Jonas <[email protected]>
@backportbot backportbot bot requested a review from mejo- July 29, 2025 15:42
@backportbot backportbot bot added bug Something isn't working 3. to review labels Jul 29, 2025
@backportbot backportbot bot added this to the Nextcloud 30.0.14 milestone Jul 29, 2025
@mejo- mejo- changed the title [stable30] [stable31] Further TextDirection improvements [stable30] Further TextDirection improvements Jul 29, 2025
@codecov
Copy link

codecov bot commented Jul 29, 2025

Codecov Report

❌ Patch coverage is 64.70588% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.65%. Comparing base (1c91ede) to head (ab9ddf8).
⚠️ Report is 4 commits behind head on stable30.

Files with missing lines Patch % Lines
src/extensions/Keymap.js 0.00% 2 Missing ⚠️
src/nodes/Table/TableCellView.vue 33.33% 2 Missing ⚠️
src/nodes/Table/TableHeaderView.vue 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           stable30    #7517      +/-   ##
============================================
- Coverage     51.67%   51.65%   -0.03%     
============================================
  Files           117      117              
  Lines          2804     2813       +9     
  Branches        560      561       +1     
============================================
+ Hits           1449     1453       +4     
- Misses         1225     1230       +5     
  Partials        130      130              

☔ 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.

mejo- added 2 commits July 29, 2025 22:51
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 <[email protected]>
Fixes styling for details and callouts with RTL script.

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- force-pushed the backport/7515/stable30 branch from f17aaac to ab9ddf8 Compare July 29, 2025 20:59
@mejo- mejo- marked this pull request as ready for review July 29, 2025 20:59
@mejo- mejo- merged commit 8ad14ca into stable30 Jul 29, 2025
84 of 89 checks passed
@mejo- mejo- deleted the backport/7515/stable30 branch July 29, 2025 21:17
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