[Toolbar Redesign] PDF has it's bottom clipped
Categories
(Fenix :: Toolbar, defect, P2)
Tracking
(firefox128 disabled, firefox129 disabled, firefox130 disabled, firefox131 disabled, firefox132 verified)
People
(Reporter: dpop, Assigned: petru)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [avocado sprint][kitkat banana sprint])
Attachments
(7 files)
Steps to reproduce
- Open the latest Nightly 130.0a1 with the toolbar redesign enabled.
- Open a PDF file e.g. https://research.nhm.org/pdfs/10840/10840.pdf
- Tap the 3-dot menu button from the Navigation bar, and tap 'Find in Page'.
- Enter a word to search (that exists in more than one place).
- Tap the up and down buttons or tap anywhere on the screen to dismiss the keyboard.
- Observe the find in page text bar.
Expected behavior
The Find in page text bar is shown above the keyboard.
The Navigation toolbar and address bar are hidden.
Actual behavior
The Find in page text bar appears under the navigation bar, and cannot be used.
Depending on the address bar position, improper blank spaces appear at the bottom and top of the screen when the Find in page text bar is invoked.
Device information
- Firefox version: Nightly 130.0a1 from 07/18
- Android device model: Google Pixel 8 Pro
- Android OS version: Android 14
Additional information
With the additional space taken by the navigation bar, ticket 1809173 should be further investigated, since the navigation bar is currently not dynamic in PDFs.
Comment 2•3 months ago
|
||
I think this bug is the same web page issue as bug 1907751. Instead of closing this bug as a duplicate of bug 1907751, I'll it open so we remember to test the PDF Viewer.
:petru , Does this get fixed by Bug 1908620 ?
Assignee | ||
Comment 4•3 months ago
|
||
Seems like there are two issues here after entering "find in page" mode:
- the PDF does not extend to the bottom of the viewport (screenshot added from 128 release). May have more to do with bug 1908249 but I think this will require a new patch.
- the find in page bar is hidden by the navbar. Do think indeed this issue will get fixed after bug 1908620.
Keeping my NI to remember to check after the above linked tickets get fixed.
Assignee | ||
Comment 5•2 months ago
|
||
Checked now, seems like one of the above two issues have been solved - the find in page bar is not hidden anymore by the navbar.
But the issue with the pdf not extending to full page is still there.
I'll look into fixing this last issue.
Assignee | ||
Updated•2 months ago
|
Assignee | ||
Updated•2 months ago
|
Assignee | ||
Comment 6•2 months ago
|
||
An interesting detail:
- if find in page is opened in landscape and then the app is rotated to portrait there is no issue - (in landscape or portrait)
- if find in page is opened in portrait and then the app is rotated to landscape the issue is always there
=> Seems like there is a different pdf initialization process in portrait vs landscape.
Assignee | ||
Comment 7•2 months ago
|
||
Also, the remaining part - the bottom of the pdf being clipped is apparently an old bug - bug 1817810 and the cause is related to us configuring the engine view with space for dynamic toolbar but on pdf there's a bug with the toolbar not being dynamic - bug 1809173.
Assignee | ||
Comment 8•2 months ago
|
||
This will inform whether the current page is a pdf or not.
Assignee | ||
Comment 9•2 months ago
|
||
Assignee | ||
Comment 10•2 months ago
|
||
This is useful in the scenario of coming back to browsing a 100vh page after finishing
a find in page session.
When using a navbar this is happening by default since in the same scenario the navbar
is reset thus triggering an initial clipping evaluation in the appropriate behavior.
Comment 11•2 months ago
|
||
Comment 12•2 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/23917b3525d6
https://hg.mozilla.org/mozilla-central/rev/bfc966c85cdf
https://hg.mozilla.org/mozilla-central/rev/446f6d4fcb59
Comment 13•2 months ago
|
||
Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.
Assignee | ||
Updated•2 months ago
|
Comment 14•2 months ago
|
||
The patch landed in nightly and beta is affected.
:petru, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox131
towontfix
.
For more information, please visit BugBot documentation.
Assignee | ||
Comment 15•2 months ago
|
||
This depends on many other changes in flux for the navbar while being an issue possibly from the start.
Would prefer to let the fix ride the trains.
Assignee | ||
Updated•2 months ago
|
Updated•2 months ago
|
Reporter | ||
Comment 16•2 months ago
|
||
Verified as fixed on the latest Nightly 132.0a1 from 09/12.
Tested with the following devices:
- Samsung Galaxy A53 5G (Android 14)
- Xiaomi 12 Pro (Android 13)
- Samsung Tab S8 Ultra 5G (Android 14)
Updated•2 months ago
|
Description
•