Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Embedded Trykker font #3541

Merged
merged 1 commit into from
Jun 18, 2020
Merged

Embedded Trykker font #3541

merged 1 commit into from
Jun 18, 2020

Conversation

nicknezis
Copy link
Contributor

Fixes #3540 by embedding the Trykker fonts as directed on this Google webfonts helper app.

@nicknezis nicknezis added the ui label Jun 18, 2020
@nicknezis nicknezis self-assigned this Jun 18, 2020
@nicknezis
Copy link
Contributor Author

There is also an option to support only Modern browsers:

/* trykker-regular - latin-ext_latin */
@font-face {
  font-family: 'Trykker';
  font-style: normal;
  font-weight: 400;
  src: local('Trykker Regular'), local('Trykker-Regular'),
       url('../fonts/trykker-v8-latin-ext_latin-regular.woff2') format('woff2'), /* Chrome 26+, Opera 23+, Firefox 39+ */
       url('../fonts/trykker-v8-latin-ext_latin-regular.woff') format('woff'); /* Chrome 6+, Firefox 3.6+, IE 9+, Safari 5.1+ */
}

I opted for the more generic default:

@font-face {
  font-family: 'Trykker';
  font-style: normal;
  font-weight: 400;
  src: url('../fonts/trykker-v8-latin-ext_latin-regular.eot'); /* IE9 Compat Modes */
  src: local('Trykker Regular'), local('Trykker-Regular'),
       url('../fonts/trykker-v8-latin-ext_latin-regular.eot?#iefix') format('embedded-opentype'), /* IE6-IE8 */
       url('../fonts/trykker-v8-latin-ext_latin-regular.woff2') format('woff2'), /* Super Modern Browsers */
       url('../fonts/trykker-v8-latin-ext_latin-regular.woff') format('woff'), /* Modern Browsers */
       url('../fonts/trykker-v8-latin-ext_latin-regular.ttf') format('truetype'), /* Safari, Android, iOS */
       url('../fonts/trykker-v8-latin-ext_latin-regular.svg#Trykker') format('svg'); /* Legacy iOS */
}

This adds more font formats to the distribution which slightly increases the size, but it might better support older browsers. I'm happy to change the logic to only support newer browsers if you think that would be best.

@joshfischer1108 joshfischer1108 merged commit 9377b10 into master Jun 18, 2020
nicknezis added a commit that referenced this pull request Sep 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Trykker fonts do not work in secure air gap environment
3 participants