Skip to content

fix(viewer): align content of old version in viewer version comparison - #9029

Merged
mejo- merged 1 commit into
mainfrom
fix/viewer_comparison_alignment
Aug 11, 2026
Merged

fix(viewer): align content of old version in viewer version comparison#9029
mejo- merged 1 commit into
mainfrom
fix/viewer_comparison_alignment

Conversation

@mejo-

@mejo- mejo- commented Aug 11, 2026

Copy link
Copy Markdown
Member

The source viewer in the "compare side by side" view in Viewer app doesn't have a menubar, so it needs a margin-top to align both versions vertically.

🖼️ Screenshots

🏚️ Before 🏡 After
image image

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

@mejo- mejo- self-assigned this Aug 11, 2026
@mejo- mejo- added bug Something isn't working 3. to review papercut labels Aug 11, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Aug 11, 2026
@mejo- mejo- moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Aug 11, 2026
The source viewer in the "compare side by side" view in Viewer app
doesn't have a menubar, so it needs a margin-top to align both versions
vertically.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
mejo- force-pushed the fix/viewer_comparison_alignment branch from 4381ed2 to 22546be Compare August 11, 2026 09:42
@mejo-

mejo- commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

/backport to stable34

@mejo-

mejo- commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

/backport to stable33

@max-nextcloud max-nextcloud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thanks!

@mejo-
mejo- merged commit bbc6dc4 into main Aug 11, 2026
62 of 65 checks passed
@mejo-
mejo- deleted the fix/viewer_comparison_alignment branch August 11, 2026 12:15
@github-project-automation github-project-automation Bot moved this from 👀 In review to ☑️ Done in 📝 Productivity team Aug 11, 2026
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 papercut

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

2 participants