Adds a FluentCxAudio#91
Merged
Merged
Conversation
JeremyJ78
requested review from
MarvinKlein1508,
PascalVorwerk and
oneolddev
as code owners
|
✅ All tests passed successfully Details on your Workflow / Core Tests page. |
Summary - Unit Tests Code CoverageSummary
CoverageFluentUI.Blazor.Community.Components - 72.3%
|
Contributor
|
The component has some glitches:
|
Contributor
|
I have used this file for testing: |
Contributor
Author
Thanks for the file... Maybe it will help for finding few bugs and handle them |
added 2 commits
October 1, 2025 20:50
Add the metadata of the file Modify the position of the playlist Modify the compact player and floating to be better Prepare the properties
Contributor
Author
|
@MarvinKlein1508 : I've added the extraction of the metadata and correct many issues you saw... The next step will be to have a properies dialog to show most of the metadata |
MarvinKlein1508
approved these changes
Oct 30, 2025
added 2 commits
October 30, 2025 11:14
Contributor
Author
|
@MarvinKlein1508 : Done |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
📖 Description
FluentCxAudio is a part of a very big component.
But it can work as standalone
It can play, pause, stop, resume an audio file.
You can add playlist...
4 views : Default, Compact, Floating, Minimal...
Each audio track can have it's cover (url, direct url)
🎫 Issues
The HTML Audio Element has an issue when seeking... it's seems to be a known bug...
(On browser try this :
document.querySelector('audio');
after that, get the audio element and :
audio.pause();
audio.currentTime = 160; (in seconds and less than the duration)
audio.play(); <- here, the audio element doesn't seek well...
👩💻 Reviewer Notes
📑 Test Plan
✅ Checklist
General
Component-specific
⏭ Next Steps
The next step will be to add a way to get the cover inside the tags of the audio file...
Try to find why seek is not working on the HTML Audio Element if we don't use the HTML Audio element in direct