Skip to content

fix: unified search results cut off on Android browser - #493

Merged
psawalka merged 1 commit into
stable32from
nmc/5808-web-android-files-search-is-not-showing-all-results
Jul 29, 2026
Merged

fix: unified search results cut off on Android browser#493
psawalka merged 1 commit into
stable32from
nmc/5808-web-android-files-search-is-not-showing-all-results

Conversation

@suet-kei-chan

Copy link
Copy Markdown

On Android Chrome, vh units resolve against the large viewport (address bar hidden), so the modal and dialog content were sized taller than the actually visible area whenever the address bar was shown. This pushed the results list, including the "Load more results" footer, below the fold and made it unreachable when a search matched many files.

Switch the modal and unified search dialog height constraints from vh to dvh so they track the real visible viewport as the address bar shows and hides. Also remove a mobile-only override that reset the modal container's flex-direction back to row and let it scroll itself, which collapsed the vertical layout the dialog relies on for its sticky header and scrollable results region.

On Android Chrome, vh units resolve against the large viewport (address
bar hidden), so the modal and dialog content were sized taller than the
actually visible area whenever the address bar was shown. This pushed
the results list, including the "Load more results" footer, below the
fold and made it unreachable when a search matched many files.

Switch the modal and unified search dialog height constraints from vh
to dvh so they track the real visible viewport as the address bar
shows and hides. Also remove a mobile-only override that reset the
modal container's flex-direction back to row and let it scroll itself,
which collapsed the vertical layout the dialog relies on for its
sticky header and scrollable results region.
@suet-kei-chan suet-kei-chan self-assigned this Jul 23, 2026
@psawalka
psawalka merged commit 745c23e into stable32 Jul 29, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants