-
Notifications
You must be signed in to change notification settings - Fork 226
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
Use HTML in the podcast description #3249
Conversation
Generated by 🚫 Danger |
📲 You can test the changes from this Pull Request in 📱 Mobile by scanning the QR code below to install the corresponding build.
📲 You can test the changes from this Pull Request in 🚗 Automotive by scanning the QR code below to install the corresponding build.
📲 You can test the changes from this Pull Request in ⌚ Wear by scanning the QR code below to install the corresponding build.
|
6aa2bdf
to
438ec19
Compare
438ec19
to
e448212
Compare
857b097
to
c491c2c
Compare
2dffe15
to
36d07e1
Compare
title = "Use HTML in the podcast description", | ||
defaultValue = true, | ||
tier = FeatureTier.Free, | ||
hasFirebaseRemoteFlag = true, |
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.
Since you are already setting this with the firebase flag, I am adding this comment just as a reminder to add the toggle in firebase
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.
It looks better after this change 👍🏽
Description
The server now returns a HTML formatted version of the podcast description. The podcast page has been updated to support the HTML.
There is one issue that I haven't been able to resolve. When using HTML in the view, the three dots to show that you can expand doesn't work. I’m not quite sure how to fix it yet so I will leave it for now.
Testing Instructions
Check with the feature flag off
Screenshots
Checklist
./gradlew spotlessApply
to automatically apply formatting/linting)modules/services/localization/src/main/res/values/strings.xml
I have tested any UI changes...