Skip to content
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

Merged

Conversation

pedrolamas
Copy link
Member

Mitigates the following warning:

image

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 pedrolamas@gmail.com

Signed-off-by: Pedro Lamas <pedrolamas@gmail.com>
@pedrolamas pedrolamas added the GH - Bug Something isn't working label Jul 28, 2022
@pedrolamas pedrolamas added this to the 1.19.1 milestone Jul 28, 2022
@pedrolamas pedrolamas marked this pull request as draft July 28, 2022 14:37
@kerbilg
Copy link
Member

kerbilg commented Jul 28, 2022

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

@pedrolamas
Copy link
Member Author

Given what you said on the other discussion, maybe this is a Safari issue?

@kerbilg
Copy link
Member

kerbilg commented Jul 28, 2022

Given what you said on the other discussion, maybe this is a Safari issue?

The same on Windows 🤔
I am also not alone: vuejs/vue-cli#3269

kerbilg
kerbilg previously approved these changes Aug 1, 2022
Copy link
Member

@kerbilg kerbilg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works perfectly 👍

@pedrolamas pedrolamas marked this pull request as ready for review August 1, 2022 10:19
@pedrolamas
Copy link
Member Author

Cool, marking as ready for review then! 🙂

matmen
matmen previously approved these changes Aug 1, 2022
Copy link
Member

@matmen matmen left a 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..

src/App.vue Outdated Show resolved Hide resolved
@pedrolamas pedrolamas dismissed stale reviews from matmen and kerbilg via faa8e01 August 1, 2022 21:52
@pedrolamas pedrolamas requested a review from matmen August 1, 2022 21:54
@pedrolamas pedrolamas merged commit aa55234 into fluidd-core:develop Aug 2, 2022
@pedrolamas pedrolamas deleted the pedrolamas/console-warning branch August 2, 2022 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GH - Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants