Skip to content

Reduce transcript banner drop shadow elevation#5302

Open
lromero16 wants to merge 1 commit into
mainfrom
fix/transcript-banner-shadow-too-heavy
Open

Reduce transcript banner drop shadow elevation#5302
lromero16 wants to merge 1 commit into
mainfrom
fix/transcript-banner-shadow-too-heavy

Conversation

@lromero16
Copy link
Copy Markdown
Collaborator

Description

Fixes #PCDROID-398

Reduces the drop shadow on the "View Transcript" banner from 2.dp to 1.dp to make it less visually heavy, consistent with other small card elements in the app.

Testing Instructions

  1. Open an episode that has a transcript available
  2. Verify the "View Transcript" banner has a lighter, more subtle drop shadow

Screenshots or Screencast

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

Reduce shadow elevation from 2dp to 1dp on the View Transcript banner
to make it less visually heavy.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lromero16 lromero16 requested a review from a team as a code owner May 13, 2026 19:52
@lromero16 lromero16 requested review from MiSikora and removed request for a team May 13, 2026 19:52
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@dangermattic
Copy link
Copy Markdown
Collaborator

1 Error
🚫 PR requires a [Type] label and either a [Area] or [Project] label.
1 Warning
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

@geekygecko geekygecko requested review from geekygecko and removed request for MiSikora May 14, 2026 04:37
Copy link
Copy Markdown
Member

@geekygecko geekygecko left a comment

Choose a reason for hiding this comment

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

I believe the Compose shadow has improved in recent versions would it be possible to closer match the design?

Image

Spec

Image

Implementation

Image

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.

4 participants