Best practice feature request: touch and wheel event listeners as `passive`

[PolGuixe]

Feature request

Description

Would it be possible to have a prop to enable/disable the touch and wheel event listeners as passive?

It is recommended by Lighthouse

More info here

Browser Affected

• Chrome
• Edge
• Firefox
• Helios
• IE
• Safari

OS

• Android
• iOS
• Linux
• MacOS
• Windows

Versions

• model-viewer: v0.10.0
• three.js: r115

[PolGuixe]

The warning disapears in Lighthouse if you are using the NPM package, instead of the Unpkg ones.

[elalish]

That is quite strange. We can't mark these as passive because they call preventDefault(); it'd be strange if your page scrolled when you moved the model. Lighthouse says it filters out these listeners, but it's clearly failing. I'll file a bug on them.