Refactor voice messages touch interaction

[alfogrillo]

This PR enhance the touch interaction with voice messages:

• User can tap on the waveform directly to seek
• User can drag the cursor to seek
• Vertical scroll in the timeline and swipe to reply work nicely

Result

int-demo.mp4

[github-actions]

	Warnings
⚠️	Please add a changelog.
⚠️	Some of the commits are missing ticket numbers. Please consider squashing all commits that don't have a tracking number.
⚠️	You seem to have made changes to views. Please consider adding screenshots.
⚠️	ElementX/Sources/Screens/RoomScreen/RoomScreenViewModel.swift#L352 - Arguments can be omitted when matching enums with associated values if they are not used. (empty_enum_arguments)
⚠️	ElementX/Sources/Screens/RoomScreen/RoomScreenViewModel.swift#L1086 - File should contain 1000 lines or less: currently contains 1086 (file_length)

Generated by 🚫 Danger Swift against 7f3ffed

[github-actions]

📱 Scan the QR code below to install the build for this PR.
🔒 This build is for internal testing purpose. Only devices listed in the ad-hoc provisioning profile can install Element Alpha.

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

[codecov]

Codecov Report

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

Comparison is base (2eb8945) 69.05% compared to head (a960974) 69.07%.
Report is 1 commits behind head on develop.

❗ Current head a960974 differs from pull request most recent head 7f3ffed. Consider uploading reports for the commit 7f3ffed to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1970      +/-   ##
===========================================
+ Coverage    69.05%   69.07%   +0.02%     
===========================================
  Files          516      517       +1     
  Lines        35347    35319      -28     
  Branches     17184    17147      -37     
===========================================
- Hits         24408    24398      -10     
+ Misses       10300    10285      -15     
+ Partials       639      636       -3     

Flag	Coverage Δ
unittests	54.31% <72.00%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...ources/Other/Progress/ProgressCursorModifier.swift	100.00% <100.00%> (ø)
...Screens/ComposerToolbar/View/ComposerToolbar.swift	76.57% <100.00%> (ø)
...oserToolbar/View/VoiceMessagePreviewComposer.swift	70.32% <100.00%> (+7.82%)	⬆️
.../Sources/Screens/RoomScreen/RoomScreenModels.swift	71.79% <ø> (ø)
...urces/Screens/RoomScreen/RoomScreenViewModel.swift	49.40% <ø> (+0.05%)	⬆️
.../RoomScreen/View/Style/LongPressWithFeedback.swift	83.33% <100.00%> (+2.08%)	⬆️
...s/VoiceMessages/VoiceMessageRoomTimelineView.swift	53.33% <ø> (+2.26%)	⬆️
...een/View/Style/TimelineItemBubbledStylerView.swift	96.39% <0.00%> (-0.22%)	⬇️
...creen/View/Style/TimelineItemPlainStylerView.swift	92.05% <0.00%> (ø)
...s/VoiceMessages/VoiceMessageRoomPlaybackView.swift	74.02% <63.63%> (+25.30%)	⬆️
... and 1 more

... and 1 file with indirect coverage changes

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

[callumu]

This all works well, thanks for making the changes.

One small request - can you turn off the haptic on scrubbing? I want to add haptics in the next version, but not when the user is scrubbing through a voice message

[alfogrillo]

This all works well, thanks for making the changes.

One small request - can you turn off the haptic on scrubbing? I want to add haptics in the next version, but not when the user is scrubbing through a voice message

Sure, consider it done! 👍🏻

[pixlwave]

Looks great to me 👍

[nimau]

Nice, it's definitely better to have the vertical scrolling working 👍

[nimau]

LGTM

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

No Coverage information
1.8% Duplication