BREAKING CHANGE: Remove distribution of fonts

youseedk / dna

CSS framework for yousee.dk

ISC License

6 stars 6 forks source link

BREAKING CHANGE: Remove distribution of fonts #299

Closed grandorf closed 3 years ago

grandorf commented 3 years ago

This one has high priority because we are distributing our fonts which we are not allowed to.

havgry commented 3 years ago

@razvandk //s.c.dk/fonts/*, I guess. I see that the font-face rules are still pointing to that. I think this is just a reaction to outright distributing it in DNA.

grandorf commented 3 years ago

@razvandk //s.c.dk/fonts/*, I guess. I see that the font-face rules are still pointing to that. I think this is just a reaction to outright distributing it in DNA.

Exactly. The DNA components are using CDN. We just added the fonts as an extra feature.

grandorf commented 3 years ago

Looks okay for now. I understand if you want this merged quickly. I would like this to be followed up by something in the documentation... Anything... That outlines the changes from version 2 to 3.

I will make sure to highlight this change in GitHub and in our changelog https://dna.yousee.dk/docs/changelog.