Rule: Prefer `.instanceOf(T)` instead of `instanceof T` for class instance checks

[Fevol]

When using pop-out windows, event and primitive prototypes will differ from the main window, causing instanceof checks to fail in rare cases. When checking for HTMLElement, MouseEvent, etc., it is always preferable to use the .instanceOf(T) variant, over the regular instanceof T.

[saberzero1]

Is there a situation where the other option is preferable?

[Fevol]

Is there a situation where the other option is preferable?

As far as I understand, no. Though I should clarify my original message: instanceof T should only be used for prototypes from packages (so instanceof TFile should still be used).

Basically, everything that derives from the base Node or UIEvent DOM classes should always use .instanceOf(T). I'm not sure if there is a handy list for it available somewhere, as I imagine that there are quite a few classes/interfaces that would match.

[saberzero1]

Relevant: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/instanceof#instanceof_and_multiple_realms