Re: [whatwg/fullscreen] Add keyboard locking option (PR #232)

saschanaz left a comment (whatwg/fullscreen#232)

> Why? `navigationUI` seems like it doesn't need to be exposed.

Because I want it to be future-ready, so that if we add anything more in the options bag then we can add it there.

I think it should be okay to just omit some of the options. (I don't fully understand why exposing `navigationUI` would be harmful though.)

> Good point, maybe the API should act like it's always available even if there's no keyboard. Then I guess we also don't want to reject in `requestFullscreen`.

Agreed.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fullscreen/pull/232?email_source=notifications&email_token=ACLYVMKP2JCCOMYSVKQK37D4XDUOFA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMRZG44DKMZXGQ22M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOS2XA4S7MNXW23LFNZ2F633QMVXF6Y3MNFRWW#issuecomment-4297853745
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fullscreen/pull/232/c4297853745@github.com>