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
[bug] The site doesn't render with firefox addon "BP Privacy Block All Font and Glyph Detection" enable. #1372
Comments
Thanks for the report :) |
did some investigation on this. The code triggering the extension is part of monaco. I haven't looked into why monaco is doing this, but I suspect it isn't for telemetry purposes. However, the actual issue is caused by the extension. The offending line in the extensions source code: for (var allowed of document.fonts.values())
if (normalised == allowed) return true This code just simply doesn't work in firefox. for (var allowed of document.fonts.values()) console.log(allowed); This, however, does work: for (var allowed of document.fonts) console.log(allowed); I suggest you contact the extension author so they can fix the issue if you intend to continue using it. I think this issue can be closed (or upstreamed with monaco if you desire). |
Ok, I will contact the extention author. You caneclore this issue.
Envoyé depuis ProtonMail mobile
…-------- Message d'origine --------
On 15 juil. 2019 à 1:21 PM, Austin Morton a écrit :
did some investigation on this.
The code triggering the extension is [part of monaco](https://github.com/microsoft/vscode/blob/013501950e78b9dde5c2e6ec3f2ddfb9201156b7/src/vs/editor/browser/config/charWidthReader.ts). I haven't looked into why monaco is doing this, but I suspect it isn't for telemetry purposes.
However, the actual issue is caused by the extension.
I don't know if the extension would otherwise break the site if it was working correctly, but the extension is simply broken.
The offending line in the extensions source code:
for
(
var
allowed
of
document
.
fonts
.
values
())
if
(normalised
==
allowed)
return
true
This code just simply doesn't work in firefox.
Try pasting this into the firefox console:
for
(
var
allowed
of
document
.
fonts
.
values
())
console
.
log
(allowed);
[image](https://user-images.githubusercontent.com/63636/61212574-b44f1200-a6d0-11e9-953b-aefb2b694b70.png)
This, however, does work:
for
(
var
allowed
of
document
.
fonts
)
console
.
log
(allowed);
[image](https://user-images.githubusercontent.com/63636/61212614-cb8dff80-a6d0-11e9-8232-3e4e23cf9c09.png)
I suggest you contact the extension author so they can fix the issue if you intend to continue using it.
I think this issue can be closed (or upstreamed with monaco if you desire).
—
You are receiving this because you authored the thread.
Reply to this email directly, [view it on GitHub](#1372), or [mute the thread](https://github.com/notifications/unsubscribe-auth/ABPJZY7QCGGHHYCAD2OTTNDP7RMUJANCNFSM4HLCJKKA).
|
Everything is in the title. If you enable BP Privacy Block All Font and Glyph Detection addon on firefox, you will see that only the top banner is displayed.
Firefox 66.0.4 - 64 bits - archlinux (up-to-date as of today)
The following error can be found in the console:
The text was updated successfully, but these errors were encountered: