Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jul 23, 2025

Backport of #7474

Todo

  • Review and resolve any conflicts
  • 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 31.0.8 milestone Jul 23, 2025
@mejo- mejo- force-pushed the backport/7474/stable31 branch from 0379583 to 0d2d3e6 Compare July 26, 2025 10:12
@mejo- mejo- marked this pull request as ready for review July 26, 2025 10:17
@mejo- mejo- force-pushed the backport/7474/stable31 branch 2 times, most recently from 31fe112 to 059ca56 Compare July 26, 2025 10:22
@codecov
Copy link

codecov bot commented Jul 26, 2025

Codecov Report

❌ Patch coverage is 50.76923% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.34%. Comparing base (c38a77a) to head (254a5a2).
⚠️ Report is 3 commits behind head on stable31.

Files with missing lines Patch % Lines
src/helpers/links.js 31.57% 13 Missing ⚠️
src/plugins/LinkBubblePluginView.js 0.00% 8 Missing ⚠️
src/components/Editor/PreviewOptions.vue 63.63% 4 Missing ⚠️
src/editor.js 0.00% 4 Missing ⚠️
src/components/Link/LinkBubbleView.vue 57.14% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##           stable31    #7476   +/-   ##
=========================================
  Coverage     58.34%   58.34%           
=========================================
  Files           290      290           
  Lines         36341    36391   +50     
  Branches        840      840           
=========================================
+ Hits          21204    21234   +30     
- Misses        15017    15037   +20     
  Partials        120      120           

☔ 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:49
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 <jonas@freesources.org>
Fixes: #1226

Signed-off-by: Jonas <jonas@freesources.org>
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 <jonas@freesources.org>
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/7474/stable31 branch from 059ca56 to 254a5a2 Compare July 26, 2025 10:49
@mejo- mejo- merged commit 4e0cbd1 into stable31 Jul 26, 2025
63 of 64 checks passed
@mejo- mejo- deleted the backport/7474/stable31 branch July 26, 2025 11:01
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