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

[Jetcaster] Handling of HTML Elements in PodcastSummary on Foldable Devices #1339

Merged

Conversation

yongsuk44
Copy link
Contributor

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

before after
HTML-TAG html-refactor
@yongsuk44 yongsuk44 requested a review from a team as a code owner April 12, 2024 06:17
@yongsuk44 yongsuk44 requested a review from arriolac April 12, 2024 06:17
@arriolac
Copy link
Contributor

arriolac commented Apr 12, 2024

Thanks! I'm aware of this issue but I think it would be good to use the new parseAsHtml method in 1.7.0-alpha06. Since Jetcaster doesn't depend on 1.7.0 yet, this is a good interim solution.

As mentioned in #1338, we have been working on jetcaster/all_form_factors as the soon-to-be state of Jetcaster on main. Would you mind changing the base branch of this PR to jetcaster/all_form_factors instead?

@yongsuk44 yongsuk44 changed the base branch from main to jetcaster/all_form_factors April 12, 2024 16:41
@yongsuk44
Copy link
Contributor Author

Would you mind changing the base branch of this PR to jetcaster/all_form_factors instead?

@arriolac Done :)

@arriolac
Copy link
Contributor

Looks like there are some conflicts. Please address

…er/html-txt

# Conflicts:
#	Jetcaster/app/src/main/java/com/example/jetcaster/ui/player/PlayerScreen.kt
@yongsuk44
Copy link
Contributor Author

sry, I found the conflicting parts and have made the corrections. 🥲

@arriolac arriolac merged commit 8550dce into android:jetcaster/all_form_factors Apr 12, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants