Skip to content

Conversation

@nmerget
Copy link
Collaborator

@nmerget nmerget commented Apr 16, 2025

No description provided.

Copy link
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 ensures that typography settings correctly apply font weights by adding the "fontWeight" property.

  • Added "fontWeight" to the typographyKeys array.
  • Initialized "fontWeight" in the updatedValue object within the traverseSABTypography function.
Files not reviewed (7)
  • package.json: Language not supported
  • src/components/Customization/Settings/ColorSelection/ColorPicker/index.scss: Language not supported
  • src/data/default-theme/typography/expressive.json: Language not supported
  • src/data/default-theme/typography/functional.json: Language not supported
  • src/data/default-theme/typography/regular.json: Language not supported
  • src/index.scss: Language not supported
  • src/tailwind.css: Language not supported

@mfranzke mfranzke self-assigned this Apr 17, 2025
@nmerget nmerget merged commit 662b817 into main Apr 17, 2025
5 checks passed
@nmerget nmerget deleted the chore-update-fonts branch April 17, 2025 06:49
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in UX Engineering Team Backlog Apr 17, 2025
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.

3 participants