Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jul 23, 2025

Backport of #7474

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits
  • Amend HEAD commit to remove the line stating to skip CI

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

@backportbot backportbot bot requested review from max-nextcloud and mejo- July 23, 2025 16:23
@backportbot backportbot bot added bug Something isn't working 3. to review feature: links labels Jul 23, 2025
@backportbot backportbot bot added this to the Nextcloud 30.0.14 milestone Jul 23, 2025
@mejo- mejo- force-pushed the backport/7474/stable30 branch 2 times, most recently from e177ec1 to 9d7c496 Compare July 26, 2025 10:22
@mejo- mejo- marked this pull request as ready for review July 26, 2025 10:22
@codecov
Copy link

codecov bot commented Jul 26, 2025

Codecov Report

❌ Patch coverage is 26.66667% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.00%. Comparing base (8796bde) to head (2a6cf30).
⚠️ Report is 11 commits behind head on stable30.

Files with missing lines Patch % Lines
src/helpers/links.js 11.11% 7 Missing and 1 partial ⚠️
src/plugins/LinkBubblePluginView.js 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           stable30    #7475      +/-   ##
============================================
- Coverage     51.32%   51.00%   -0.33%     
============================================
  Files           116      116              
  Lines          2712     2737      +25     
  Branches        544      550       +6     
============================================
+ Hits           1392     1396       +4     
- Misses         1191     1211      +20     
- Partials        129      130       +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.

mejo- added 7 commits July 26, 2025 12:54
We have a custom node for displaying images anyway.

This fixes the cursor being a pointer in the preview images of preview
links.

Signed-off-by: Jonas <[email protected]>
When initiating the editor extensions (i.e. in `addProseMirrorPlugins()`),
`editor.contentComponent` is not yet available. So let's read it when we
actually use it, when creating the tooltip in `LinkBubblePluginView`.

Before, a new Vue instance was initiated for each `LinkBubbleView.vue`
as the `parent` given to `VueRenderer()` was `undefined`. With this fix,
`LinkBubbleView.vue` becomes a child component of the root editor Vue
instance.

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- force-pushed the backport/7474/stable30 branch from 9d7c496 to 2a6cf30 Compare July 26, 2025 10:54
@mejo- mejo- merged commit 32536e4 into stable30 Jul 26, 2025
61 of 63 checks passed
@mejo- mejo- deleted the backport/7474/stable30 branch July 26, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants