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

Migrate to Material 3 (BC break!) #313

Merged
merged 4 commits into from
Jan 13, 2023
Merged

Migrate to Material 3 (BC break!) #313

merged 4 commits into from
Jan 13, 2023

Conversation

hrach
Copy link
Contributor

@hrach hrach commented Jan 13, 2023

No description provided.

@hrach hrach added the feature New feature or request label Jan 13, 2023
@hrach hrach self-assigned this Jan 13, 2023
This was linked to issues Jan 13, 2023
this fixes Tabs usage in Orbit after migration to M3
@hrach hrach marked this pull request as ready for review January 13, 2023 15:02
@hrach hrach requested a review from a team as a code owner January 13, 2023 15:02
@hrach hrach merged commit e02048e into main Jan 13, 2023
@hrach hrach deleted the m3 branch January 13, 2023 15:19
Copy link
Contributor

@Drekorian Drekorian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small issues detected, please fix in a follow-up merge request.

}
}

@Preview
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Render problem: java.lang.IllegalStateException: Only Sp can convert to Px

@Preview
@Composable
private fun TagScreenPreview() {
    AppTheme {
        TabsScreen {}
    }
}

defaultFontFamily = title1.fontFamily ?: FontFamily.Default,
h1 = title1.copy(fontSize = 96.sp, lineHeight = TextUnit.Unspecified),
h2 = title1.copy(fontSize = 60.sp, lineHeight = TextUnit.Unspecified),
h3 = title1.copy(fontSize = 38.sp, lineHeihgt = TextUnit.Unspecified),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: lineHeight.

@hrach hrach changed the title Migrate to Material 3 Migrate to Material 3 (BC break!) Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate to Material 3 Tabs
3 participants