-
Notifications
You must be signed in to change notification settings - Fork 3
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
Not working on Microsoft Edge browser. #2
Comments
ok found the problem, I was using dark mode extension, checking background colour is not reliable because extensions like, I use or chrome dark mode flag may interfere with these. |
Yes, I even made a PR to bypass that problem but @HelloZeroNet never accepted it. |
Here is an example for easier check: elem_local_test = createElem('<img src="http://127.0.0.1:43110/favicon.ico?random=' + Date.now() + '" style="display: none"/>');
elem_local_test.onerror = function() {
document.getElementById("visit-links").classList.add("no-local");
}; https://github.com/HelloZeroNet/zeronet.io/blob/master/go.html#L68 |
@krzysztof113 what do you think about the @HelloZeroNet Suggestion ? |
Yes, I think I'll move to it. The only problem I've seen is when you have regular webserver on custom port then favicon will be there, with false positive, but the chance for that is negligible. |
site not detecting ZeroNet installation on new Microsoft edge browser.
The text was updated successfully, but these errors were encountered: