Skip to content

feat(new-default-font): public sans #95

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 23, 2024
Merged

Conversation

josh-chamberlain
Copy link
Contributor

@josh-chamberlain josh-chamberlain commented Jul 9, 2024

Fixes

fixes #96

Contents

  • adds Public Sans to the top of the list as a default; retains Libre Franklin
  • adds the fonts.googleapis embed to index.html, I was seeing a system font

Future PRs

  • In order for this change to take effect, the <head> of the index.html of anything we want to update will need to change, as well as bumping the design system version. I can handle it, we just need a new design system version first.
    • graceful deprecation: adding a fonts.googleapis embed for Public Sans, and leaving the Libre Franklin one. That way, no matter which version of the design system is used, one of those two fonts will be used (user settings notwithstanding)
    • less graceful: swapping them out to cut down on page load, and making sure a recent version of the design system is used. I think this will be OK though
      • part of why I wanted to update both main and beta just to get this over with
      • if this is troublesome, we can just wait until v2

Docs/related

  • I mentioned this in the brand assets:
Screen Shot 2024-07-09 at 2 02 01 PM

@josh-chamberlain josh-chamberlain changed the base branch from main to beta July 18, 2024 14:12
Copy link

gitguardian bot commented Jul 18, 2024

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
9480360 Triggered Generic Password 0435567 src/components/Form/form.spec.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@josh-chamberlain josh-chamberlain merged commit c4faab8 into beta Jul 23, 2024
5 checks passed
@josh-chamberlain josh-chamberlain deleted the fix-public-sans branch July 23, 2024 15:15
Copy link

🎉 This PR is included in version 3.0.0-beta.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Update to public sans
2 participants