Skip to content

Fix bottom padding for HTML text in Wear module#1697

Merged
kul3r4 merged 4 commits into
android:mainfrom
kul3r4:updates
Jun 18, 2026
Merged

Fix bottom padding for HTML text in Wear module#1697
kul3r4 merged 4 commits into
android:mainfrom
kul3r4:updates

Conversation

@kul3r4

@kul3r4 kul3r4 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Fix bottom padding for HTML text in Wear module, required changing the HTMLTextContainer which is used also on phone sample.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a modifier parameter to HtmlTextContainer to allow custom styling, which is then utilized in EpisodeScreen.kt to apply vertical content padding to the episode summary paragraphs. However, a logic bug was identified in EpisodeScreen.kt where the entire summary string is passed to HtmlTextContainer instead of the individual paragraph item, which would cause the full summary to be rendered repeatedly for each paragraph.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@kul3r4 kul3r4 marked this pull request as ready for review June 18, 2026 13:10
@kul3r4 kul3r4 requested review from garanj and riggaroo June 18, 2026 13:12
@kul3r4 kul3r4 changed the title Fix bottom padding for HTML text Fix bottom padding for HTML text in Wear module Jun 18, 2026
@kul3r4 kul3r4 removed the request for review from riggaroo June 18, 2026 21:13
@kul3r4 kul3r4 merged commit 6488a6d into android:main Jun 18, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants