Skip to content
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

fix: using theme in Text's components, renaming typescales to fonts #3351

Merged
merged 5 commits into from
Sep 22, 2022

Conversation

lukewalczak
Copy link
Member

@lukewalczak lukewalczak commented Sep 5, 2022

Summary

Fixes: #3305 #3369

PR:

  • renames typescale to fonts to avoid confusion between them in version 5
  • correct Text component to accept fontFamily
  • enable the possibility to use theme fonts in components using Text with variant, such as:
    • AppbarContent

    • BottomNavigation

    • Chip

    • DialogTitle

    • DrawerCollapsedItem

    • FABGroup

    • MenuItem

    • SegmentedButtonItem

    • TextInput

Test plan

  • added tests for configureFonts
  • update snapshots

@callstack-bot
Copy link

callstack-bot commented Sep 5, 2022

Hey @lukewalczak, thank you for your pull request 🤗. The documentation from this branch can be viewed here.

@github-actions
Copy link

github-actions bot commented Sep 5, 2022

The mobile version of example app from this branch is ready! You can see it here

.

1 similar comment
@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@bsubedi26
Copy link

@lukewalczak When are you planning on merging this?

@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@hosseinmd
Copy link

LGTM. Please merge

@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@lukewalczak lukewalczak changed the title fix: correct Text, unblock using theme in TextInput fix: using theme in Text's components, renaming typescales to fonts Sep 22, 2022
@lukewalczak lukewalczak merged commit 002ea0f into main Sep 22, 2022
@lukewalczak lukewalczak deleted the fix/correct-typescales branch September 22, 2022 11:10
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.

Font Family cannot be recognised in theme
4 participants