Skip to content

global requires a polyfill. Shall we replace global with globalThis? #129

@brendonjohn

Description

@brendonjohn

Hey there, I love the library.

From testing a library in stripped-down environments I've realised react-shadow assumes that global is available for accessing the global this value.

Do you have a suggested polyfill for making global available?

Alternatively, would you accept a PR to replace the use of global with globalThis? I'm happy to send the PR through.

image

https://caniuse.com/mdn-javascript_builtins_globalthis

This should be significantly better without needing to make an invasive change.

🙇

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions