Skip to content

Swap to using Inter fonts#295

Merged
cigamit merged 2 commits intomainfrom
Inter-font
Apr 9, 2026
Merged

Swap to using Inter fonts#295
cigamit merged 2 commits intomainfrom
Inter-font

Conversation

@cigamit
Copy link
Copy Markdown
Contributor

@cigamit cigamit commented Apr 9, 2026

This brings us more in line with other CIQ applications, and moves away from the RH fonts.

@cigamit cigamit requested a review from TheWitness April 9, 2026 05:08
@cigamit cigamit self-assigned this Apr 9, 2026
@cigamit cigamit added the enhancement New feature or request label Apr 9, 2026
Copilot AI review requested due to automatic review settings April 9, 2026 05:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates AWX’s UI typography to use the Inter font family instead of the prior Red Hat/Open Sans font setup, aligning the UI styling with other CIQ applications.

Changes:

  • Switch global UI font-family to Inter and adjust some Patternfly title styling overrides.
  • Update the “installing” (migration) page to reference Inter font assets.
  • Update custom error page CSS to use Inter .woff2 fonts (and add Inter font assets under awx/static).

Reviewed changes

Copilot reviewed 3 out of 19 changed files in this pull request and generated 4 comments.

File Description
awx/ui/src/ascender.css Introduces Inter @font-face and applies Inter to body/Patternfly titles.
awx/ui/public/installing.html Updates font references on the installing/migration page.
awx/static/Inter-Regular.woff2 Adds Inter Regular font asset for server-rendered/static pages.
awx/static/custom_error.css Switches custom error pages from Red Hat TTF fonts to Inter WOFF2 fonts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread awx/ui/src/ascender.css
Comment thread awx/ui/src/ascender.css
Comment thread awx/ui/public/installing.html
Comment thread awx/static/custom_error.css
@Klaas-
Copy link
Copy Markdown
Contributor

Klaas- commented Apr 9, 2026

do you need to add a license of the font to the repo?

https://github.com/rsms/inter/blob/master/LICENSE.txt maybe?

@cigamit
Copy link
Copy Markdown
Contributor Author

cigamit commented Apr 9, 2026

Added. I had to add a separate folder for it, as the tests fail if we chunk it in the UI, as its not an npm module.

@cigamit cigamit merged commit 2f3780c into main Apr 9, 2026
@cigamit cigamit deleted the Inter-font branch April 9, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

4 participants