-
-
Notifications
You must be signed in to change notification settings - Fork 434
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
fix: mitigates console warning #800
fix: mitigates console warning #800
Conversation
Signed-off-by: Pedro Lamas <[email protected]>
Fresh installed Edge on macos. I have built fluidd and host it with worldwideweb 🤔 Very weird, I will test it on windows later. Bildschirmaufnahme.2022-07-28.um.16.47.52.mov |
Given what you said on the other discussion, maybe this is a Safari issue? |
The same on Windows 🤔 |
Signed-off-by: Pedro Lamas <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works perfectly 👍
Cool, marking as ready for review then! 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code wise this LGTM (though see comment); but I can't currently test this for functionality..
Signed-off-by: Pedro Lamas <[email protected]>
Mitigates the following warning:
From my tests with Edge and Chrome on Windows 11, it seems to work fine with the value back in!
This is a follow up on this discussion: #774 (comment)
Signed-off-by: Pedro Lamas [email protected]