Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Add internal playback of audio files #3207

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

awesome-manuel
Copy link
Contributor

@awesome-manuel awesome-manuel commented Jul 9, 2019

This is a draft implementation of internal audio playback.

Pull Request Checklist

  • Changes has been tested on an Android device or Android emulator with API 16
  • UI change has been tested on both light and dark themes
  • Pull request is based on the develop branch
  • Pull request updates CHANGES.rst
  • Pull request includes screenshots or videos if containing UI changes
  • Pull request includes a sign off

@intsven
Copy link

intsven commented Sep 2, 2019

Here are some screenshots of how this looks:
Screenshot_20190902-154556_AMPchat
Screenshot_20190902-154607_AMPchat
Screenshot_20190902-154649_AMPchat

@bmarty
Copy link
Member

bmarty commented Sep 2, 2019

Hi @awesome-manuel ,
Thanks for the PR. After a quick review, the code LGTM. I will check at runtime.
FTR the core team is working more on RiotX now, but if all is Ok, I will merge the PR anyway.

@bmarty
Copy link
Member

bmarty commented Sep 2, 2019

Pb I see so far:

  • mp3 playback does not work. I haven't tested other format yet.
  • When scrolling, the button state (play/pause) is not restored

@intsven
Copy link

intsven commented Sep 3, 2019

When scrolling, the button state (play/pause) is not restored

What do you mean by that? When I scroll everything just keeps working like you would expect it

@intsven
Copy link

intsven commented Sep 3, 2019

mp3 playback does not work. I haven't tested other format yet.

I usually use audio/mp4. Do we really need mp3 support because the Android MediaRecorder does not even support recording mp3?

@awesome-manuel
Copy link
Contributor Author

Maybe fall back to download in case of MP3?

@intsven
Copy link

intsven commented Sep 4, 2019

I have tested mp3 now and it works for me inline just as the other audio messages (m4a, ...).
I tested this file: https://ia802508.us.archive.org/5/items/testmp3testfile/mpthreetest.mp3

@awesome-manuel
Copy link
Contributor Author

Can someone explain what this means exactly?

$ ./tools/check/check_code_quality.sh
Check drawable quantity
ERROR, missing drawable alternative.

Signed-off-by: Manuel Stahl <manuel.stahl@awesome-technologies.de>
Change-Id: If753ce84a66d5c1ac5d176267dccb16c147cf6de
@DoM1niC
Copy link

DoM1niC commented Oct 10, 2019

@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.

@awesome-manuel
Copy link
Contributor Author

@bmarty any chance we can get this merged at all?

@ghost
Copy link

ghost commented Feb 15, 2020

Sorry if this is not the correct place to ask, but does this mean this will be included in the next version?

@tuxayo
Copy link
Contributor

tuxayo commented Mar 18, 2020

Is there anything to take here for RiotX? At least the design principles?

@tuxayo
Copy link
Contributor

tuxayo commented Mar 18, 2020

RiotX issue: element-hq/element-android#29

@LarzlRael
Copy link

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants