-
Notifications
You must be signed in to change notification settings - Fork 395
Add internal playback of audio files #3207
base: develop
Are you sure you want to change the base?
Add internal playback of audio files #3207
Conversation
e97eaca
to
c0807ab
Compare
c0807ab
to
52c6928
Compare
Hi @awesome-manuel , |
Pb I see so far:
|
What do you mean by that? When I scroll everything just keeps working like you would expect it |
I usually use audio/mp4. Do we really need mp3 support because the Android MediaRecorder does not even support recording mp3? |
Maybe fall back to download in case of MP3? |
I have tested mp3 now and it works for me inline just as the other audio messages (m4a, ...). |
Can someone explain what this means exactly?
|
52c6928
to
271bb88
Compare
Signed-off-by: Manuel Stahl <manuel.stahl@awesome-technologies.de> Change-Id: If753ce84a66d5c1ac5d176267dccb16c147cf6de
271bb88
to
4343a9a
Compare
@awesome-manuel Could you add Support 4 .ogx files like WhatsApp ? It won't playback currently only over a Third Party Media Player possible, I think .ogx is Opus based Audiofile in my mind. |
@bmarty any chance we can get this merged at all? |
Sorry if this is not the correct place to ask, but does this mean this will be included in the next version? |
Is there anything to take here for RiotX? At least the design principles? |
RiotX issue: element-hq/element-android#29 |
the play and pause icons appear even when the files are not audio, also it does not play all some formats like mp3 p m4a |
This is a draft implementation of internal audio playback.
Pull Request Checklist