Skip to content

feat: Add metadata and MediaSession handlers to enable native controls#313

Open
noman-land wants to merge 5 commits intomasterfrom
audio-player
Open

feat: Add metadata and MediaSession handlers to enable native controls#313
noman-land wants to merge 5 commits intomasterfrom
audio-player

Conversation

@noman-land
Copy link
Owner

@noman-land noman-land commented Apr 7, 2025

Closes: #172

@noman-land noman-land added the enhancement New feature or request label Apr 7, 2025
@noman-land noman-land requested a review from Copilot April 7, 2025 04:00
@noman-land noman-land self-assigned this Apr 7, 2025
@noman-land noman-land moved this to In Progress in transcript.fish Apr 7, 2025
@noman-land noman-land added this to the Public beta milestone Apr 7, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

src/js/audio/audioUtils.ts:7

  • [nitpick] If the Episode type guarantees that 'episode' and 'title' are defined, consider removing the optional chaining for clarity.
title: `${episode?.episode}: ${episode?.title}`,

src/js/audio/AudioContextProvider.tsx:65

  • [nitpick] The type check for 'episodeIdx' is redundant since findIndex always returns a number; consider removing it for cleaner code.
if (typeof episodeIdx === 'number') {

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 7, 2025

Deploying transcript-fish with  Cloudflare Pages  Cloudflare Pages

Latest commit: 84a7ddc
Status: ✅  Deploy successful!
Preview URL: https://ee117429.transcript-fish.pages.dev
Branch Preview URL: https://audio-player.transcript-fish.pages.dev

View logs

@noman-land noman-land added typescript Updates to Typescript code ux Updates to user experience audio Updates to how audio is played mobile labels Apr 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audio Updates to how audio is played enhancement New feature or request mobile typescript Updates to Typescript code ux Updates to user experience

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

feat(audio): Make native media buttons work

1 participant