-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[Jetcaster] Handling of HTML Elements in PodcastSummary on Foldable Devices #1339
[Jetcaster] Handling of HTML Elements in PodcastSummary on Foldable Devices #1339
Conversation
Updated depedencies
Unnecessary state reads for colors, which never change. Following official [M3 color scheme](https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/ColorScheme.kt?q=androidx.compose.material3.ColorScheme)
…avigationBar (android#1323) Fixed Issue : android#1322 ![Screenshot_20240407_154318](https://github.com/android/compose-samples/assets/38935359/1fbefe71-bfe3-4e24-b8f2-7d810b6a865f)
Thanks! I'm aware of this issue but I think it would be good to use the new As mentioned in #1338, we have been working on |
@arriolac Done :) |
Looks like there are some conflicts. Please address |
…er/html-txt # Conflicts: # Jetcaster/app/src/main/java/com/example/jetcaster/ui/player/PlayerScreen.kt
sry, I found the conflicting parts and have made the corrections. 🥲 |
Jetcaster/app/src/main/java/com/example/jetcaster/ui/player/PlayerScreen.kt
Show resolved
Hide resolved
8550dce
into
android:jetcaster/all_form_factors
There are cases where the Summary Text does not contain HTML elements. In these instances, it is considered that the performance overhead is negligible, therefore no additional processing was implemented