Skip to content

Dialog v1: Fix title font family - #4246

Merged
siddharthkp merged 4 commits into
mainfrom
dialog-v1-fix-font-family
Feb 12, 2024
Merged

Dialog v1: Fix title font family#4246
siddharthkp merged 4 commits into
mainfrom
dialog-v1-fix-font-family

Conversation

@siddharthkp

Copy link
Copy Markdown
Member

Not sure about the historical context of this code, but I think we can let the Dialog get it styles by inherit them

@changeset-bot

This comment was marked as outdated.

@siddharthkpsiddharthkp self-assigned this Feb 9, 2024
@siddharthkpsiddharthkp added the update snapshots 🤖 Command that updates VRT snapshots on the pull request label Feb 9, 2024
@github-actions

github-actionsBot commented Feb 9, 2024

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize
packages/react/dist/browser.esm.js113.37 KB (-0.02% 🔽)
packages/react/dist/browser.umd.js114.02 KB (-0.02% 🔽)

@github-actionsgithub-actionsBot removed the update snapshots 🤖 Command that updates VRT snapshots on the pull request label Feb 9, 2024
@github-actions
github-actionsBottemporarily deployed to storybook-preview-4246 February 9, 2024 16:33 Inactive
@github-actions
github-actionsBottemporarily deployed to storybook-preview-4246 February 12, 2024 15:46 Inactive
@siddharthkp
siddharthkp marked this pull request as ready for review February 12, 2024 15:53
@siddharthkp
siddharthkp requested a review from a teamFebruary 12, 2024 15:53
@siddharthkp
siddharthkp added this pull request to the merge queueFeb 12, 2024
Merged via the queue into main with commit 2aced1cFeb 12, 2024
@siddharthkp
siddharthkp deleted the dialog-v1-fix-font-family branch February 12, 2024 16:02
@primerprimerBot mentioned this pull request Feb 12, 2024
lukasoppermann pushed a commit that referenced this pull request Apr 16, 2024
* don't need to set fontFamily, inherit it
* Create hot-hounds-speak.md
* test(vrt): update snapshots
---------
Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The Text component in Dialog v1 is not properly rendering font-family

2 participants

@siddharthkp@joshblack