Skip to content

fix(fuselage): AudioPlayer loading and seeking states#1857

Open
durgeshbg wants to merge 4 commits intoRocketChat:mainfrom
durgeshbg:fix/audio-player-loading-and-seeking-states
Open

fix(fuselage): AudioPlayer loading and seeking states#1857
durgeshbg wants to merge 4 commits intoRocketChat:mainfrom
durgeshbg:fix/audio-player-loading-and-seeking-states

Conversation

@durgeshbg
Copy link

@durgeshbg durgeshbg commented Feb 1, 2026

fix(fuselage): AudioPlayer loading and seeking states

Proposed changes (including videos or screenshots)

Fixed the initial state of the Audio Player when the media file is in loading state by introducing a loading state and rending a loading icon with disabled controls. This change will improve the user experience by informing them to wait before interacting with the player.

Screen.Recording.2026-02-01.at.9.07.01.PM.mov

These changes also fix the seek feature in the player which lags to update when in pause/play state this is due to the browser trying to load the audio file to the specific time. This has been fixed via introducing another state variable and handling it via dom events of the audio tag to display a loading state and disables the controls as above to let the user's wait.

Improved stories by adding a new story and added units tests for the component which were absent previously.

Screen.Recording.2026-02-01.at.9.06.01.PM.mov

Issue(s)

Fixes: #1856

Further comments

@changeset-bot
Copy link

changeset-bot bot commented Feb 1, 2026

⚠️ No Changeset found

Latest commit: 1f412b8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@CLAassistant
Copy link

CLAassistant commented Feb 1, 2026

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

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.

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.

AudioPlayer: Seek action lag and no optimistic UI causing bad UX

2 participants