Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: adapt audio recording wave form to the available space #2435

Merged
merged 3 commits into from
Jun 27, 2024

Conversation

MartinCupela
Copy link
Contributor

🎯 Goal

Adjust the number of bars and the gap width based on the available space.

🛠 Implementation details

The solution uses ResizeObserver as the only way to react to size change of the root div. The window resize would be useful only to mobile device screen rotation, but not possible message list resizing if implemented by integrators.

🎨 UI Changes

6Y0O.webm

Copy link

codecov bot commented Jun 25, 2024

Codecov Report

Attention: Patch coverage is 95.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 85.48%. Comparing base (2ca3188) to head (710e271).
Report is 2 commits behind head on release-v11.

Files Patch % Lines
...mponents/Attachment/components/WaveProgressBar.tsx 93.75% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           release-v11    #2435      +/-   ##
===============================================
+ Coverage        85.44%   85.48%   +0.03%     
===============================================
  Files              392      393       +1     
  Lines             9078     9121      +43     
  Branches          2345     2355      +10     
===============================================
+ Hits              7757     7797      +40     
- Misses            1120     1122       +2     
- Partials           201      202       +1     

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

@MartinCupela MartinCupela merged commit aed0360 into release-v11 Jun 27, 2024
7 of 8 checks passed
@MartinCupela MartinCupela deleted the fix/resize-wave-form branch June 27, 2024 15:04
github-actions bot pushed a commit that referenced this pull request Jun 28, 2024
## [11.23.0](v11.22.0...v11.23.0) (2024-06-28)

### Bug Fixes

* adapt audio recording wave form to the available space ([#2435](#2435)) ([aed0360](aed0360))
* always load thread replies on Thread opening ([#2436](#2436)) ([579953c](579953c))
* forward groupStyles to Message in VirtualizedMessageList ([#2440](#2440)) ([241f5d7](241f5d7))
* reflect correctly the translation key in Timestamp component ([741e9ce](741e9ce))

### Features

* configure message group size by max time between messages ([#2439](#2439)) ([0d094cb](0d094cb))
@stream-ci-bot
Copy link
Collaborator

🎉 This PR is included in version 11.23.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

4 participants