Skip to content

Fix: #21432 - Add padding to voice-to-content BottomSheet #21928

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

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

Osbaldo-Arellano
Copy link

@Osbaldo-Arellano Osbaldo-Arellano commented May 29, 2025

Title

Fix voice-to-content BottomSheet UI overlap on small screens


Issue

Fixes #21432


Description

This PR addresses the issue where the voice-to-content BottomSheet is partially obscured on smaller screens. I think the root cause was missing navigation bar padding, causing UI elements to be obscured. The fix adds .navigationBarsPadding() to the BottomSheet container.

Due to the voice-to-content feature being inaccessible in the dev build, full voice-to-content testing has not yet been possible. I plan to test the changes with the CI build once available.


Question for reviewers:

Is there a recommended way to enable or test the voice-to-content feature in the dev build environment?


Testing Instructions

  1. Launch the app on a device or emulator with a small screen or gesture navigation enabled.
  2. Enable voice_to_content in Debug Settings > Remote Features
  3. Tap the FAB on My Site
  4. Select "Post from audio"
  5. Verify that the BottomSheet content is fully visible within the display area without clipping or overlap.

Labels

  • [Type] Bug
  • UI
  • Good First Issue

Thank you for reviewing!

Copy link

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.

Voice to content overlap
1 participant