Skip to content

fix(editor): fix viewer header overlaying page content on mobile - #8952

Merged
mejo- merged 1 commit into
mainfrom
fix/mobile_overlay
Aug 3, 2026
Merged

fix(editor): fix viewer header overlaying page content on mobile#8952
mejo- merged 1 commit into
mainfrom
fix/mobile_overlay

Conversation

@mejo-

@mejo- mejo- commented Aug 3, 2026

Copy link
Copy Markdown
Member

The rule resulted in the editor content being covered partially by the viewer header when opening a shared document on real mobile browsers (e.g. Firefox on Android).

I was not able to reproduce the issue with mobile view in Firefox on desktop. Via remote debugging on Android I was able to pin the issue to this rule though.

Fixes: #8709

Reverts 87f2ddf. The issue that this commit originally aimed to fix (superfluous scrollbar with short content on mobile) is also no longer reproducible on mobile browsers.

🏁 Checklist

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

The rule resulted in the editor content being covered partially by the
viewer header when opening a shared document on real mobile browsers
(e.g. Firefox on Android).

I was not able to reproduce the issue with mobile view in Firefox on
desktop. Via remote debugging on Android I was able to pin the issue to
this rule though.

Fixes: #8709

Reverts 87f2ddf. The issue that this
commit originally aimed to fix (superfluous scrollbar with short content
on mobile) is also no longer reproducible on mobile browsers.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- self-assigned this Aug 3, 2026
@mejo- mejo- added bug Something isn't working 3. to review labels Aug 3, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Aug 3, 2026
@mejo- mejo- moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Aug 3, 2026
@mejo-

mejo- commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

/backport to stable34

@grnd-alt
grnd-alt self-requested a review August 3, 2026 14:57
@mejo-
mejo- merged commit a8ba381 into main Aug 3, 2026
60 of 65 checks passed
@mejo-
mejo- deleted the fix/mobile_overlay branch August 3, 2026 15:47
@github-project-automation github-project-automation Bot moved this from 👀 In review to ☑️ Done in 📝 Productivity team Aug 3, 2026
@mejo-

mejo- commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

/backport to stable33

@mejo-

mejo- commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

/backport to stable32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

[Bug]: Mobile overlay issue on .md text files on all major browsers

2 participants