Skip to content

feat: swap fonts; use Cormorant; reduce size of large variant#157

Draft
domsteinbach wants to merge 4 commits intomainfrom
150-replace-serif-font
Draft

feat: swap fonts; use Cormorant; reduce size of large variant#157
domsteinbach wants to merge 4 commits intomainfrom
150-replace-serif-font

Conversation

@domsteinbach
Copy link
Copy Markdown
Collaborator

No description provided.

@domsteinbach domsteinbach linked an issue Nov 27, 2025 that may be closed by this pull request
@flicksolutions
Copy link
Copy Markdown
Member

@domsteinbach
This replaces all fonts with Cormorant. In the meeting, we specified to only change the font of the headings. TH stated that a sans serif font for the content is important and state-of-the-art. Did MS specifically request the same serif font throughout the website?

@domsteinbach
Copy link
Copy Markdown
Collaborator Author

In my conversations it was about the Fassungen view only. I noted that it will be easier, faster, more consistent, better maintainable ... when we replace all at once and do not use different fonts for different texts - except for transcriptions which would be ok of course. But having the headers in a different font than other text ... I can not support that.

I will have a look together with MS and get back to you.

@domsteinbach
Copy link
Copy Markdown
Collaborator Author

@flicksolutions I just had a video call. We can go for replacing all with this font. It is a step forward for MS. But we might find even a better one.

Copy link
Copy Markdown
Member

@flicksolutions flicksolutions left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You set the name to Cormorant Garamond so you will have to use this name everywhere. The path was right before your latest commit.

--base-font-color: var(--color-surface-950);
--base-font-color-dark: var(--color-surface-50);
--base-font-family: system-ui;
--base-font-family: inherit;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change doesn't really make sense. A base font-family should be set here..
(and the change does nothing because this file is nowhere imported and not used)

Suggested change
--base-font-family: inherit;

@domsteinbach domsteinbach marked this pull request as draft December 17, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

replace serif font

2 participants