Skip to content

feat: set default typography - #16

Merged
andrey-canon merged 1 commit into
open-release/verawood.nelpfrom
and/FUTUREX-1670
Aug 19, 2026
Merged

andrey-canon merged 1 commit into
open-release/verawood.nelpfrom
and/FUTUREX-1670

Conversation

@andrey-canon

Copy link
Copy Markdown

Description

This add the default font for the NELC implementation.

Issue # 1670
Migration pr of #13 and #14

How to test

  1. Set the learning MFE as a volume
  2. Add the module.config.js file into the frontend-app-learning folder and restart your container
module.exports = {
    localModules: [
        { moduleName: '@edx/brand', dir: '../brand-openedx'},
    ],
};
  1. Checkout this branch
  2. In redwood the styles loaded automatically due to this file however those lines were removed in this version so you have to import the brand package explicitly in the frontend-app-learning/src/index.scss file, add the following lines at the top of the file after the last import statement.
@import "@edx/brand/paragon/fonts";
@import "@edx/brand/paragon/variables";
@import "@edx/brand/paragon/overrides";

Before

image

After

image

This set the IBM Plex Sans Arabic font as default.

@johanseto johanseto left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

working in my local env

Image

@andrey-canon
andrey-canon merged commit ba8dc23 into open-release/verawood.nelp Aug 19, 2026
1 check passed
@andrey-canon
andrey-canon deployed to open-release/verawood.nelp August 19, 2026 21:44 — with GitHub Actions Active
@andrey-canon
andrey-canon deployed to open-release/verawood.nelp August 19, 2026 21:45 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
open-release/verawood.nelp — b00a5e6d Deployed Aug 19, 2026 by andrey-canon via create-jira-issue / create_jira_issue #13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants