a way for the rotate instructions interstitial to announce its show/hide-ing

[gmarinov]

e.g. document.body.dispatchEvent(new CustomEvent('VRInstructionsShown'))

[borismus]

Why? How are the rotate instructions not desirable?

On Thu, Apr 14, 2016 at 12:23 PM Georgi Marinov notifications@github.com
wrote:

e.g. document.body.dispatchEvent(new CustomEvent('VRInstructionsShown'))

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#93

[gmarinov]

it's desirable to sync to the interstitial becoming frontmost or hiding...

[kschzt]

@borismus it's not about desirability, this is just an event to know when it's shown / hidden

[borismus]

This can be done, propose an event name, and we can emit it. This is already done for other events like onvrdisplaypresentchange

[gmarinov]

well, it's up there at the very top of the post. if it needs to be in all lowercase, then that would make it vrinstructionsshown, and its counterpart - vrinstructionshidden.

[borismus]

How about having the vrdisplaychange event fire after the instructions are
hidden?

On Fri, May 6, 2016 at 2:57 AM Georgi Marinov notifications@github.com
wrote:

well, it's up there at the very top of the post. if it needs to be in all
lowercase, then that would make it "vrinstructionsshown", and its
counterpart - "vrinstructionshidden".

—
You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
#93 (comment)

[gmarinov]

how would that address the 'shown' part?

[borismus]

It would not.

On Fri, May 6, 2016 at 8:40 AM Georgi Marinov notifications@github.com
wrote:

how would that address the 'shown' part?

—
You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
#93 (comment)

[gmarinov]

so can we get a vrinstructionsshown and vrinstructionshidden then?

[borismus]

Sure, please submit a PR.

On Fri, May 6, 2016 at 8:51 AM Georgi Marinov notifications@github.com
wrote:

so can we get a vrinstructionsshown and vrinstructionshidden then?

—
You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
#93 (comment)

[borismus]

Out of scope. Polyfill needs to track WebVR spec.