-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update design tokens #305
Update design tokens #305
Conversation
📱 New catalog for testing generated: Download |
@@ -125,7 +125,9 @@ | |||
<item name="tabsLabelFont">?font_family_medium</item> | |||
<item name="linkFont">?font_family_medium</item> | |||
<item name="title1Font">?font_family_medium</item> | |||
<item name="title2Font">?font_family_regular</item> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title2 in Movistar is bold
@@ -138,7 +140,9 @@ | |||
<item name="tabsLabelTextStyle">normal</item> | |||
<item name="linkTextStyle">normal</item> | |||
<item name="title1TextStyle">normal</item> | |||
<item name="title2TextStyle">normal</item> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
light
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those parameters are correct. Android just check if the style of the text is bold, italic or normal.
Normal means, no bold, no italic.
What causes the text to use the light font is "title2Font" attribute.
Sounds weird I know, but thanks for noticing! 😇
@@ -138,7 +140,9 @@ | |||
<item name="tabsLabelTextStyle">normal</item> | |||
<item name="linkTextStyle">normal</item> | |||
<item name="title1TextStyle">normal</item> | |||
<item name="title2TextStyle">normal</item> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Light
<item name="indicatorTextStyle">normal</item> | ||
<item name="navigationBarTextStyle">normal</item> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Medium
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This PR was automatically created by the import-design-tokens GitHub Action.