search

sindresorhus / screenfull

Simple wrapper for cross-browser usage of the JavaScript Fullscreen API
https://sindresorhus.com/screenfull
MIT License
7.08k stars 698 forks source link

Add support for options #182

Closed silltho closed 3 years ago

silltho commented 4 years ago

I needed the support of the FullscreenOptions to hide the navbar on mobile devices. Since the PR #148 related to the issue was closed I wanted to share this with you.

resolves #114

IssueHunt Summary ### Referenced issues This pull request has been submitted to: - [#114: Support `requestFullscreen()` options](https://issuehunt.io/repos/3362916/issues/114) ---
silltho commented 4 years ago

summoning @sindresorhus 😄

sindresorhus commented 3 years ago

You need to add it to the readme too.

silltho commented 3 years ago

Good point. I updated the api documentation and added an example how to hide navigation ui on mobile devices.

sindresorhus commented 3 years ago

You need to update the .toggle() docs in the readme too.