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

[Feature] Adjust voice-recording seconds remaining notification to 30 seconds and limit to 5min #8776

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

Conversation

OdinVex
Copy link

@OdinVex OdinVex commented Mar 12, 2024

Signed-off-by: Odin Vex 44311901+OdinVex@users.noreply.github.com

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other : Adjustment for accessibility

Content

The default voice-recording "seconds remaining" notification only shows up at the last 10sec of a clip and that can cause panic and anxiety at not being able to summarize or catch up to the point being made or discussed in a voice clip.

Users with accessibility needs such as slower talking/communicating/thinking and others with complex issues to talk about are extremely frustrated with the default 2min length of time.

Motivation and context

The notification showing up at the last 10sec is frustratingly short for a number of our users. I've adjusted the notification to show up with 30sec remaining (this seem a sweet-spot for catching a breath, thinking, then voicing to wrap up or prepare for another clip without dragging). I've also adjusted the default voice clip length limit to 5min to give people plenty of time to send a message. These fixes were desperately wanted on our Matrix server so I started supplying APKs to them through our own distribution methods so they can use it until this hopefully makes it mainline.

Main motivation: Accessibility - Our users needed these changes and I feel they'd benefit many.

Tests

  • Determined the variables used in determining the time remaining and when to show the notification.

  • Adjusted time-remaining logic to trigger at 30sec instead of 10sec.

  • Tested notification toast successfully (it is not tied to a specific voice-clip length).

  • Determined the variables used in determining the length of voice-clips.

  • Adjusted VOICE_MESSAGE_LIMIT_MS appropriately to a more usable 5min limit (up from 2min).

  • Tested successfully the voice-clip length limit by recording my own dialogue to another user along with a lot of fan noise. ^_^;

Tested devices

  • Physical
  • Emulator
  • OS version(s): Android 13

Checklist

Comments

Edit: I didn't include a changelog file because this was not done for a pre-existing Pull Request or Issue.

Sign-off is located at the beginning and bottom of this Pull Request, the Contributing Guide mentions it can be a comment in the Pull Request. (I used GitHub's Web Editor to keep things clean as I despise Android Studio and the horrific nightmare it is to use it.)

I can make adjustments if needed.

Signed-off-by: Odin Vex 44311901+OdinVex@users.noreply.github.com

10 seconds is barely noticed until you're part-way through it, 30 seconds could help users with accessibility issues.
Voice clips as short as 2min can be frustrating for some users with accessibility issues and is rather short in general, a 5min limit is much more reasonable and still not taxing on resources.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant