RoyDefined / WebDoomer

WebDoomer is a fast and efficient Zandronum and QZandronum server browser as a web app.
GNU General Public License v3.0
4 stars 0 forks source link

Main website container overflows vertically on mobile #16

Closed RoyDefined closed 6 months ago

RoyDefined commented 7 months ago

Describe the bug The main container specifying the height of the website overflows due to the browser UI taking up part of the expected height (see bottom of image in screen shot where the footer is only partially shown)

The issue is the use of h-screen max-h-screen in the main container which corresponds to heights using the vh notation, which corresponds to view height: https://github.com/RoyDefined/WebDoomer/blob/09e2beb63d585c1cee263d337416b754a138851f/src/WebDoomerClient/src/app/layout/layout.component.ts#L10

Instead, the code should use either h-svh max-h-svh for small view height, or h-dvh max-h-dvh for dynamic view height. These notations account for the adjustments in view height that occur on mobile. The question is if dynamic view height looks good and adjusts well, or if small view height has to be used instead, assuming it doesn't suddenly leave empty space.

Steps to reproduce Enter the app on mobile.

Expected behavior The website shows the full site on mobile without excess scrolling.

Screenshots bea6248f-acf0-4d74-99ee-97c79d554183

Additional context N/A

RoyDefined commented 6 months ago

Fixed in 8e06cdf.