Skip to content

Commit ee34607

Browse files
authored
FEATURE: remove optical sizing for now as well and keep ccmp on default. (#19)
ccmp composes chars so A⃝ into an A with a circle around it. Inter recommends this is default enabled and Microsoft do as well. Odds of hitting edge cases in the wild are rare given: https://github.com/rsms/inter/blob/master/src/features/ccmp.fea Additional, optical sizing is a stylistic choice, it can hurt readability on small text but make it better on big text. Let's leave this out for now. Inter website recommends liga 1, but this advise seems out of date as no new browser is impacted by whatever bug it was.
1 parent f67e632 commit ee34607

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

lib/discourse_fonts.rb

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# frozen_string_literal: true
22

33
module DiscourseFonts
4-
VERSION = "0.0.16"
4+
VERSION = "0.0.17"
55

66
def self.path_for_fonts
77
File.expand_path("../../vendor/assets/fonts", __FILE__)
@@ -55,8 +55,9 @@ def self.fonts
5555
{
5656
name: "Inter",
5757
stack: "Inter, Arial, sans-serif",
58-
font_feature_settings: "'calt' 0, 'ccmp' 0",
59-
font_variation_settings: "'opsz' 28",
58+
font_feature_settings: "'calt' 0",
59+
# calt is enabled by default, but we need to disable it so it does no conflict with markdown typographer which
60+
# performs similar oprations.
6061
# Inter is variable font, so the same file is used for all weights.
6162
variants: [
6263
{

0 commit comments

Comments
 (0)