Skip to content

Commit db5b335

Browse files
authored
Switch to using @fontsource for Inter & Inconsolata (element-hq#28540)
* Switch to using @fontsource for Inter & Inconsolata Signed-off-by: Michael Telatynski <[email protected]> * Remove unused font Open_Sans Signed-off-by: Michael Telatynski <[email protected]> * Switch to less broken imports Signed-off-by: Michael Telatynski <[email protected]> * Switch to Ubuntu 24.04 Signed-off-by: Michael Telatynski <[email protected]> * Discard changes to res/themes/dark/css/dark.pcss * Discard changes to res/themes/legacy-light/css/_fonts.pcss * Discard changes to res/themes/light-high-contrast/css/light-high-contrast.pcss * Discard changes to res/themes/light/css/light.pcss * Discard changes to .github/workflows/end-to-end-tests.yaml * Set outputDir for fonts Signed-off-by: Michael Telatynski <[email protected]> * Use headed mode for Playwright Signed-off-by: Michael Telatynski <[email protected]> * Switch to new Chrome headless mode instead Signed-off-by: Michael Telatynski <[email protected]> * Cache bust playwright browser install in CI Signed-off-by: Michael Telatynski <[email protected]> * Update screenshots Signed-off-by: Michael Telatynski <[email protected]> * Try with 22.04 Signed-off-by: Michael Telatynski <[email protected]> * Update to Ubuntu Noble Signed-off-by: Michael Telatynski <[email protected]> --------- Signed-off-by: Michael Telatynski <[email protected]>
1 parent 1d51323 commit db5b335

File tree

174 files changed

+40
-352
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

174 files changed

+40
-352
lines changed

.github/workflows/end-to-end-tests.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ jobs:
8383
name: "Run Tests ${{ matrix.runner }}/${{ strategy.job-total }}"
8484
needs: build
8585
if: inputs.skip != true
86-
runs-on: ubuntu-22.04
86+
runs-on: ubuntu-24.04
8787
permissions:
8888
actions: read
8989
issues: read
@@ -124,11 +124,11 @@ jobs:
124124
with:
125125
path: |
126126
~/.cache/ms-playwright
127-
key: ${{ runner.os }}-playwright-${{ steps.playwright.outputs.version }}
127+
key: ${{ runner.os }}-playwright-${{ steps.playwright.outputs.version }}-chromium
128128

129-
- name: Install Playwright browsers
129+
- name: Install Playwright browser
130130
if: steps.playwright-cache.outputs.cache-hit != 'true'
131-
run: yarn playwright install --with-deps
131+
run: yarn playwright install --with-deps --no-shell chromium
132132

133133
- name: Run Playwright tests
134134
run: yarn playwright test --shard ${{ matrix.runner }}/${{ strategy.job-total }}

package.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,8 @@
7979
},
8080
"dependencies": {
8181
"@babel/runtime": "^7.12.5",
82+
"@fontsource/inconsolata": "^5",
83+
"@fontsource/inter": "^5",
8284
"@formatjs/intl-segmenter": "^11.5.7",
8385
"@matrix-org/analytics-events": "^0.29.0",
8486
"@matrix-org/emojibase-bindings": "^1.3.3",

playwright.config.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,12 @@ SPDX-License-Identifier: AGPL-3.0-only OR GPL-3.0-only
66
Please see LICENSE files in the repository root for full details.
77
*/
88

9-
import { defineConfig } from "@playwright/test";
9+
import { defineConfig, devices } from "@playwright/test";
1010

1111
const baseURL = process.env["BASE_URL"] ?? "http://localhost:8080";
1212

1313
export default defineConfig({
14+
projects: [{ name: "Chrome", use: { ...devices["Desktop Chrome"], channel: "chromium" } }],
1415
use: {
1516
viewport: { width: 1280, height: 720 },
1617
ignoreHTTPSErrors: true,

playwright/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM mcr.microsoft.com/playwright:v1.49.0-jammy
1+
FROM mcr.microsoft.com/playwright:v1.49.0-noble
22

33
WORKDIR /work
44

Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.

res/fonts/Inter/Inter-Bold.woff

-140 KB
Binary file not shown.

res/fonts/Inter/Inter-Bold.woff2

-103 KB
Binary file not shown.

res/fonts/Inter/Inter-BoldItalic.woff

-147 KB
Binary file not shown.
-109 KB
Binary file not shown.

res/fonts/Inter/Inter-Italic.woff

-141 KB
Binary file not shown.

res/fonts/Inter/Inter-Italic.woff2

-104 KB
Binary file not shown.

res/fonts/Inter/Inter-Medium.woff

-139 KB
Binary file not shown.

res/fonts/Inter/Inter-Medium.woff2

-103 KB
Binary file not shown.
-147 KB
Binary file not shown.
-110 KB
Binary file not shown.

res/fonts/Inter/Inter-Regular.woff

-131 KB
Binary file not shown.

res/fonts/Inter/Inter-Regular.woff2

-96.5 KB
Binary file not shown.

res/fonts/Inter/Inter-SemiBold.woff

-139 KB
Binary file not shown.

res/fonts/Inter/Inter-SemiBold.woff2

-103 KB
Binary file not shown.
-148 KB
Binary file not shown.
-109 KB
Binary file not shown.

res/fonts/Open_Sans/LICENSE.txt

Lines changed: 0 additions & 202 deletions
This file was deleted.

res/fonts/Open_Sans/OpenSans-Bold.ttf

-219 KB
Binary file not shown.
-208 KB
Binary file not shown.
-208 KB
Binary file not shown.
-212 KB
Binary file not shown.
-216 KB
Binary file not shown.
-208 KB
Binary file not shown.

0 commit comments

Comments
 (0)