WordPress-Android icon indicating copy to clipboard operation
WordPress-Android copied to clipboard

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

Open Osbaldo-Arellano opened this issue 7 months ago • 1 comments

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!

Osbaldo-Arellano avatar May 29 '25 23:05 Osbaldo-Arellano

@Osbaldo-Arellano Thank you so much for your contribution and sorry for the delay in reviewing the PR. The issue was already fixed here, right before your PR was sent. Proceeding to close this.

adalpari avatar Jul 28 '25 11:07 adalpari