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.

Sign up for GitHub

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

Jump to bottom

Copy and Reference All Downloadable Fonts to OUJS Pro/Dev Server #227

Closed
Martii opened this issue Jun 29, 2014 · 4 comments
Closed

Copy and Reference All Downloadable Fonts to OUJS Pro/Dev Server #227

Martii opened this issue Jun 29, 2014 · 4 comments
Labels
bug You've guessed it... this means a bug is reported. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. security Usually relates to something critical. UI Pertains inclusively to the User Interface.

Comments

@Martii
Copy link
Member

Martii commented Jun 29, 2014

  1. Not thrilled that I have to (and any visitor/user) allow cloudflare and googleusercontent to the whitelist of NoScript
  2. Just now it took 4 page reloads to show the icons... this isn't good for our site.

May apply to #186 too.
@sizzlemctwizzle
Copy link
Member

I don't know why we're using CDNs for our hosting of client side JS
libraries either. I'd prefer to host everything ourselves.

@Martii
Copy link
Member Author

Martii commented Jun 29, 2014

Is it possible to use the 2nd drone as our CDN? Do we have a virtual domain name capability for that? e.g. maybe https://cdn.openuserjs.org This might explain the defer attributes I found too... this issue should solve that and possibly #148

@sizzlemctwizzle
Copy link
Member

Is it possible to use the 2nd drone as our CDN?

That would decrease overall performance and is unnecessary since traffic is distributed across them (non-blocking i/o and the way drones work). They're just text files (we even used the minified versions) and are compressed efficiently, plus express implements cache control for static files. It's not like we're talking about hosting videos here.

@Martii Martii mentioned this issue Jul 3, 2014
Merged
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Jul 9, 2014
Copy and rereferenced all downloadable fonts to pro/dev
ce08ea6 
* Leaving fontawesome CSS uncompressed for the time being... this is better to find and fix any bugs on a specific line rather than all one one single line.
* Dereferenced googleusercontent font and added to bootstrap.css

Applies to OpenUserJS#227
@Martii Martii mentioned this issue Jul 9, 2014
Merged
@Martii Martii self-assigned this Jul 9, 2014
@Martii
Copy link
Member Author

Martii commented Jul 12, 2014

Closed by #248

@Martii Martii closed this as completed Jul 12, 2014
@Martii Martii removed their assignment Jul 12, 2014
@Martii Martii mentioned this issue Jul 18, 2014
Merged
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Nov 18, 2015
Update static fonts
18a57d0 
* Add missing bootstrap .woff2 to static route for more current browsers
* Google has core ttf now only ... ref: https://fonts.gstatic.com/s/squadaone/v5/DIbfqh10Zkwc_Qd08Y0saS3USBnSvpkopQaUR-2r7iU.ttf
* Create .woff with FontForge *(Generate Fonts...)* from ttf ... smaller than original
* Create .woff2 with https://github.com/google/woff2 from ttf ... pretty good compression here
* Create .eot with `$ sudo npm install ttf2eot -g` and run their commandline with current *node*
* Create .svg with FontForge ... not compressible in current toolkit

* File rename to their current naming schema
* Draft specs at https://drafts.csswg.org/css-fonts-3/#src-desc for format hint ... ttf/otf both fall under `opentype` according to the specs and it is SIL Open Font but google of course isn't... defer to theirs... we can blame them for any breakage. ;)

**NOTES**
* Should be useful to save some transfer bandwidth in browsers that support it
* PaleMoon is still .woff compatible so need that... SeaMonkey 2.39 PASS with .woff2
* Older browsers don't support WOFF/WOFF2 at all so assuming that's why only the ttf exists
* Same licensing
* Cover some older browsers/devices with eot and svg

Followup from OpenUserJS#227
@Martii Martii mentioned this issue Nov 18, 2015
Merged
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Nov 18, 2015
Try changing preferred order for font loading
4b4aaa0 
* Currently ttf was chosen on a clean profile/no-cache... the browser might be FIFO on these CSS3 rules

Post fix for OpenUserJS#827 and followup for OpenUserJS#227
@Martii Martii mentioned this issue Nov 18, 2015
Merged
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Nov 18, 2015
Update static font
95210ce 
* Recreate woff with https://github.com/fontello/ttf2woff ... produces smaller output

Followup from OpenUserJS#227 and post change from OpenUserJS#827
@Martii Martii mentioned this issue Nov 18, 2015
Merged
@OpenUserJS OpenUserJS locked as resolved and limited conversation to collaborators Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug You've guessed it... this means a bug is reported. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. security Usually relates to something critical. UI Pertains inclusively to the User Interface.
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sizzlemctwizzle @Martii