iframe attribute 'sandbox' should be string

[weijarz]

This attribute is defined in React is: sandbox?: string | undefined

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe#attr-sandbox

[ryansolid]

It appears we have a union of the specific allowed values. Is that a problem?

type HTMLIframeSandbox =
    | "allow-downloads-without-user-activation"
    | "allow-forms"
    | "allow-modals"
    | "allow-orientation-lock"
    | "allow-pointer-lock"
    | "allow-popups"
    | "allow-popups-to-escape-sandbox"
    | "allow-presentation"
    | "allow-same-origin"
    | "allow-scripts"
    | "allow-storage-access-by-user-activation"
    | "allow-top-navigation"
    | "allow-top-navigation-by-user-activation";

[lxsmnsyc]

It is indeed a problem. Here's an example:

https://github.com/LXSMNSYC/solid-headless/blob/4b0694bd57ae90e2ead5ed06718293ca34745aba/docs/src/components/Preview.tsx#L9-L15

For now I'm ignoring this error.

[atk]

Unfortunately, a composed string literal type is too complex for typescript, so allowing string would indeed be the best option.