-
Notifications
You must be signed in to change notification settings - Fork 464
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Shadowing of Image
constructor
#3268
Comments
Indeed, do you have other in mind, so that we can do them in a single pass |
@bobzhang Not sure if this is complete list but should cover most of the cases. I printed const COPIED_WINDOW_OBJECT_STRING = `
AppView: class extends
ExtensionOptions: class extends
ExtensionView: class extends
alert: ƒ alert()
...
`;
COPIED_WINDOW_OBJECT_STRING
.split("\n")
.map(x => x.split(":")[0])
.filter(x => x && /^[A-Z]/.test(x[0]))
.join("\n") Result
|
Thanks, this seems to be huge. |
@alexfedoseev It would be great that if you can verify it is fixed in master |
I've stumbled upon this issue today. Glenn has helped me already, but posting it here just for context on another browser API binding it affects: https://stackoverflow.com/questions/54803396/module-name-is-shadowing-global-name-used-in-binding#54803396 |
I think this is solved, do you still find such issue in master or bs-platform@next? |
@bobzhang Just checked and indeed, with |
This is basically the same issue as #3214 but with
Image
constructor.The text was updated successfully, but these errors were encountered: