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

Use HTML in the podcast description #3249

Merged
merged 7 commits into from
Dec 19, 2024
Merged

Use HTML in the podcast description #3249

merged 7 commits into from
Dec 19, 2024

Conversation

geekygecko
Copy link
Member

@geekygecko geekygecko commented Nov 20, 2024

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

  1. Follow the podcast Dirtbag Rich https://pca.st/zkf1jxjd
  2. Open the podcast page
  3. Expand the header
  4. Tap on the podcast description
  5. ✅ Check the description is using HTML and split into paragraphs

Check with the feature flag off

  1. Go to Profile -> Settings -> Beta features
  2. Toggle off "Use HTML in the podcast description"
  3. Go back to the podcast page
  4. ✅ Check the description is not using HTML

Screenshots

Before After
dirtbag-rich-before Screenshot_20241218_174819

Checklist

  • If this is a user-facing change, I have added an entry in CHANGELOG.md
  • Ensure the linter passes (./gradlew spotlessApply to automatically apply formatting/linting)
  • I have considered whether it makes sense to add tests for my changes
  • All strings that need to be localized are in modules/services/localization/src/main/res/values/strings.xml
  • Any jetpack compose components I added or changed are covered by compose previews
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

I have tested any UI changes...

  • with different themes
  • with a landscape orientation
  • with the device set to have a large display and font size
  • for accessibility with TalkBack

@dangermattic
Copy link
Collaborator

dangermattic commented Nov 20, 2024

1 Warning
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Nov 20, 2024

📲 You can test the changes from this Pull Request in 📱 Mobile by scanning the QR code below to install the corresponding build.
App Name 📱 Mobile
Build TypedebugProd
Commitc4d67a8
Direct Downloadpocketcasts-app-prototype-build-pr3249-c4d67a8.apk
📲 You can test the changes from this Pull Request in 🚗 Automotive by scanning the QR code below to install the corresponding build.
App Name 🚗 Automotive
Build TypedebugProd
Commitc4d67a8
Direct Downloadpocketcasts-automotive-prototype-build-pr3249-c4d67a8.apk
📲 You can test the changes from this Pull Request in ⌚ Wear by scanning the QR code below to install the corresponding build.
App Name ⌚ Wear
Build TypedebugProd
Commitc4d67a8
Direct Downloadpocketcasts-wear-prototype-build-pr3249-c4d67a8.apk

@geekygecko geekygecko force-pushed the task/html-description branch from 6aa2bdf to 438ec19 Compare November 26, 2024 05:34
@geekygecko geekygecko force-pushed the task/html-description branch from 438ec19 to e448212 Compare December 5, 2024 00:54
@geekygecko geekygecko force-pushed the task/html-description branch 2 times, most recently from 857b097 to c491c2c Compare December 16, 2024 05:26
@geekygecko geekygecko added [Type] Enhancement Improve an existing feature. [Area] Podcast Page labels Dec 16, 2024
@geekygecko geekygecko added this to the 7.80 milestone Dec 16, 2024
@geekygecko geekygecko force-pushed the task/html-description branch from 2dffe15 to 36d07e1 Compare December 18, 2024 06:16
@geekygecko geekygecko marked this pull request as ready for review December 18, 2024 07:29
@geekygecko geekygecko requested a review from a team as a code owner December 18, 2024 07:29
@geekygecko geekygecko requested review from mebarbosa and removed request for a team December 18, 2024 07:29
title = "Use HTML in the podcast description",
defaultValue = true,
tier = FeatureTier.Free,
hasFirebaseRemoteFlag = true,
Copy link
Contributor

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

Copy link
Contributor

@mebarbosa mebarbosa left a 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 👍🏽

@geekygecko geekygecko merged commit 81eedf9 into main Dec 19, 2024
14 checks passed
@geekygecko geekygecko deleted the task/html-description branch December 19, 2024 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Area] Podcast Page [Type] Enhancement Improve an existing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants