Geo-Web-Project / browser

An interface that uses geolocation to navigate/resolve content on the Geo Web.
MIT License
3 stars 8 forks source link

feat: add full screen mode for web content #77

Closed tnrdd closed 1 year ago

tnrdd commented 1 year ago

Description

Add a full screen mode for the web content linked to the parcel and update the app layout.

Issue

fixes #75

Checklist:

Additional comments

Fixed an infinite loading issue that happened when there is an error in navigator.geolocation.getCurrentPosition, like when the user doesn't allow the browser to access the device position.

Alert Reviewers

@codynhat @gravenp

codynhat commented 1 year ago

This looks great!

The browser API to enter full screen is not supported on iOS Safari for iPhone (works for iPad), in this case only the the app UI elements are hidden.

I can confirm this is what I am experiencing on Safari for iOS, but it is still a much better experience.