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

Blotter has build issues on Firefox Nightly (FF60) & Firefox Developer (FF59) #16

Closed
bradley opened this issue Feb 15, 2018 · 3 comments
Labels

Comments

@bradley
Copy link
Owner

bradley commented Feb 15, 2018

The bug is a little intermittent and needs some investigating. On the most recent versions of Firefox it seems that if you load into http://blotter.js.org with a fresh cache the page will mostly not load Blotter elements on the first load. A refresh seems to fix it. Additionally if you load up a material page (https://blotter.js.org/#/materials/FliesMaterial) first, with a fresh cache, Blotter seems to work fine and you can navigate to the homepage with everything working as usual.

Please investigate.

Reference comment on Designer News

@bradley bradley added the bug label Feb 15, 2018
@darrenbarklie
Copy link

darrenbarklie commented Feb 15, 2018

Showing the same issue on Win FF 58.0.2.

@bradley
Copy link
Owner Author

bradley commented Feb 16, 2018

Still working on sorting this out, but definitely seems to be an issue around how github pages (http://blotter.js.org is hosted via github pages) is encoding the documentation site's javascript (br, or Brotli) and how firefox is using it prior to being cached. The problem does not seem to effect the Blotter library itself.

@bradley
Copy link
Owner Author

bradley commented Feb 18, 2018

This has been fixed. It was an issue with external font faces not being loaded before Blotter attempted to render them. On the site, the logo, nav items, and all Blotter examples use external fonts so this is why it looked like things werent working.

I don't love the fix (wrapping the Blotter instantiation calls in document.fonts.ready.then(function () { ... }); but it's just the documentation site and it works.

Bye!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants