Просмотр исходного кода

fix: unify line height across default fonts (#9513)

Marcel Mraz 2 месяцев назад
Родитель
Сommit
6b5fb30d69
2 измененных файлов с 3 добавлено и 3 удалено
  1. 2 2
      packages/common/src/font-metadata.ts
  2. 1 1
      packages/excalidraw/wysiwyg/textWysiwyg.test.tsx

+ 2 - 2
packages/common/src/font-metadata.ts

@@ -46,7 +46,7 @@ export const FONT_METADATA: Record<number, FontMetadata> = {
       unitsPerEm: 1000,
       unitsPerEm: 1000,
       ascender: 1011,
       ascender: 1011,
       descender: -353,
       descender: -353,
-      lineHeight: 1.35,
+      lineHeight: 1.25,
     },
     },
   },
   },
   [FONT_FAMILY["Lilita One"]]: {
   [FONT_FAMILY["Lilita One"]]: {
@@ -116,7 +116,7 @@ export const FONT_METADATA: Record<number, FontMetadata> = {
       unitsPerEm: 1000,
       unitsPerEm: 1000,
       ascender: 880,
       ascender: 880,
       descender: -144,
       descender: -144,
-      lineHeight: 1.15,
+      lineHeight: 1.25,
     },
     },
     fallback: true,
     fallback: true,
   },
   },

+ 1 - 1
packages/excalidraw/wysiwyg/textWysiwyg.test.tsx

@@ -1324,7 +1324,7 @@ describe("textWysiwyg", () => {
       ).toEqual(FONT_FAMILY.Nunito);
       ).toEqual(FONT_FAMILY.Nunito);
       expect(
       expect(
         (h.elements[1] as ExcalidrawTextElementWithContainer).lineHeight,
         (h.elements[1] as ExcalidrawTextElementWithContainer).lineHeight,
-      ).toEqual(1.35);
+      ).toEqual(1.25);
     });
     });
 
 
     describe("should align correctly", () => {
     describe("should align correctly", () => {