Skip to content

[stable32] fix(PreviewOptions): Move three-dot menu into preview box or link bubble - #7813

Merged
mejo- merged 2 commits into
stable32from
backport/7793/stable32
Oct 23, 2025
Merged

[stable32] fix(PreviewOptions): Move three-dot menu into preview box or link bubble#7813
mejo- merged 2 commits into
stable32from
backport/7793/stable32

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #7793

* Fixes: #6814 (broken alignment of preview options in lists)
* Fixes: #7693 (preview options displayed in read-only documents)
* Fixes: nextcloud/collectives#1920 (hidden preview options on narrow screens)
Also fixes the bug that the link is displayed above the preview in
read-only view in Collectives.
Signed-off-by: Jonas <jonas@freesources.org>
@codecov

codecovBot commented Oct 21, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 36.73469% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.95%. Comparing base (e2260ff) to head (14b679e).
⚠️ Report is 10 commits behind head on stable32.

Files with missing linesPatch %Lines
src/nodes/Preview.vue17.39%19 Missing ⚠️
src/components/Editor/PreviewOptions.vue52.63%9 Missing ⚠️
src/components/Link/LinkBubbleView.vue50.00%3 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## stable32 #7813 +/- ##
============================================
+ Coverage 53.39% 59.95% +6.56% 
============================================
Files 502 499 -3 Lines 43552 38365 -5187 Branches 1134 1090 -44 ============================================
- Hits 23254 23002 -252 + Misses 20191 15258 -4933 + Partials 107 105 -2 

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

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
mejo- merged commit 9a7d76c into stable32Oct 23, 2025
63 of 66 checks passed
@mejo-
mejo- deleted the backport/7793/stable32 branch October 23, 2025 21:03
@blizzzblizzz mentioned this pull request Nov 10, 2025
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewbugSomething isn't workingfeature: linksux

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@mejo-@marcoambrosini@benjaminfrueh