Variables for $primary-font-family und $secondary-font-family

[robinstickel]

Include more than $base-font-family

[man-oi]

Dein Vorschlag wäre also aus $base-font-family die beiden zu machen, oder zusätzlich? Spricht erstmal nichts dagegen, oft braucht man ja auch zwei

[robinstickel]

Ich glaube primary und secondary reicht, oder? Wir haben sehr selten mehr als zwei verschiedene Fonts im Einsatz, oder?

[man-oi]

Ja würde ich auch sagen, können wir gerne einführen. Überlege ob wir $base-font-family noch als Alias benutzen wollen. Also dann: $base-font-family: $primary-font-family;

[robinstickel]

Hatte ich auch überlegt, kann dann aber ggf. zu Verwirrung führen, oder? Entscheide aber gerne selbst 😄

[man-oi]

Vor allem was ist eigentlich $primary? Ist das die meist benutzte Schrift, oder die Schrift für Überschriften? Wenn $primary für Überschriften und $secondary für normalen Text wäre, dann wäre $base auch eher gleich $secondary 😄

Vielleicht sollten wir über die Benennung nochmal nachdenken? Denke gerade eher an sowas wie: $head-font-family $base-font-family

[robinstickel]

Ich würde es nicht in Headlines und normalen Text trennen. Manchmal sind sowohl Headlines als auch normaler Text in der gleichen Schriftart und nur Zitate z.B. in einer anderen. Ich sehe $primary als HAUPT Schriftart. Jene die eben am meisten genutzt wird. Und $secondary dann eben für eine "Nebenschrift". Das kann sowohl die Headline, aber auch ein Zitatmodul sein.

Vielleicht macht es dann schon Sinn sowohl $base, als auch $primary-font-family und $secondary-font-family zu machen und so zu definieren: $base-font-family: $primary oder $secondary;

[man-oi]

find ich gut $primary als main schrift zu sehen. hab das jetzt mal im branch feature/secondary-font-family umgesetzt, mit einem kleinen Kommentar für weitere font-families

[man-oi]

merged

https://github.com/me-and-company/web-project-kit/commit/4fd705f4870c10e410714d8e2a417cfba20e5ac2