Voice message bubble design improvements

[julioromano]

• Fixes color and size of both text and icons.

[julioromano]

Screenshots: https://github.com/vector-im/element-x-android/actions/runs/6642112874

[jmartinesp]

I think something broke in the screenshots? Some screenshots were deleted, others added, other apparently renamed... There's no easy way to diff the changes like this. A couple of days ago the same happened in another PR of mine. Maybe the StateProvider already has too many states and we should think about splitting it?

Could this be caused by #1636?

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[julioromano]

I think something broke in the screenshots? Some screenshots were deleted, others added, other apparently renamed... There's no easy way to diff the changes like this. A couple of days ago the same happened in another PR of mine. Maybe the StateProvider already has too many states and we should think about splitting it?

Could this be caused by #1636?

The changes are indeed there and visible in the screenshots.
Though there is also a lot of noise (as in screenshots that don't belong to something that changed in this PR).
Unfortunately you won't see any waveforms in these sshots because the library we're currently using to generate it doesn't paint on first pass so the screenshots are blank.

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/4hgUjj

[codecov]

Codecov Report

Attention: 158 lines in your changes are missing coverage. Please review.

Comparison is base (f5999a5) 63.19% compared to head (f8a06eb) 63.21%.
Report is 2 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1637      +/-   ##
===========================================
+ Coverage    63.19%   63.21%   +0.01%     
===========================================
  Files         1189     1201      +12     
  Lines        30796    31199     +403     
  Branches      6361     6446      +85     
===========================================
+ Hits         19463    19721     +258     
- Misses        8410     8518     +108     
- Partials      2923     2960      +37     

Files	Coverage Δ
...eatures/messages/impl/actionlist/ActionListView.kt	59.64% <100.00%> (+0.17%)	⬆️
...line/factories/event/TimelineItemContentFactory.kt	56.00% <100.00%> (ø)
...ures/messages/impl/timeline/groups/Groupability.kt	59.25% <100.00%> (+1.56%)	⬆️
...l/timeline/model/event/TimelineItemVoiceContent.kt	100.00% <100.00%> (ø)
...ne/model/event/TimelineItemVoiceContentProvider.kt	100.00% <100.00%> (ø)
.../impl/voicemessages/timeline/VoiceMessageEvents.kt	100.00% <100.00%> (ø)
...s/impl/voicemessages/timeline/VoiceMessageState.kt	100.00% <100.00%> (ø)
...ndroid/features/messages/impl/MessagesPresenter.kt	82.63% <0.00%> (-0.44%)	⬇️
...l/timeline/model/event/TimelineItemEventContent.kt	91.66% <50.00%> (-3.79%)	⬇️
...tils/messagesummary/MessageSummaryFormatterImpl.kt	50.00% <0.00%> (-3.34%)	⬇️
... and 11 more

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jmartinesp]

The changes in code LGTM, although I'm quite worried about the auto-incremented values in the screenshot names...