Skip to content

style: improved camera permission placemet#40

Open
iTalhaZahid wants to merge 2 commits intoahmedbna:mainfrom
iTalhaZahid:style/camera
Open

style: improved camera permission placemet#40
iTalhaZahid wants to merge 2 commits intoahmedbna:mainfrom
iTalhaZahid:style/camera

Conversation

@iTalhaZahid
Copy link

@iTalhaZahid iTalhaZahid commented Feb 20, 2026

Type of Change:

Styling Improvement

Changes made:

This PR fixes the layout and vertical alignment of the Camera Permission screen when camera access is asked.

Previously, the permission card was not properly centered on the screen and appeared misaligned vertically. The issue was caused by the permission container not being wrapped in a full-screen flex container with proper centering styles.

Changes made:

Wrapped the permission UI inside a full-screen container:

<View style={{
flex: 1,
justifyContent: "center",
alignItems: "center",
paddingHorizontal: 8,
}}>

Testing made:

  • Tested on IOS
  • Tested on Android

Screenshots:

Before

After

IOS

Android

Checklist:

  • Self-review completed
  • No breaking changes

Copilot AI review requested due to automatic review settings February 20, 2026 18:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the layout of the Camera permission request UI by ensuring the permission card is vertically/horizontally centered on screen when camera access isn’t granted.

Changes:

  • Wrapped the “Camera Access Required” permission card in a full-screen container (flex: 1) with centering styles.
  • Added horizontal padding around the centered permission content.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@iTalhaZahid iTalhaZahid marked this pull request as draft February 20, 2026 18:59
Used styles.container instead of inline styling to keep styling consistent

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@iTalhaZahid iTalhaZahid marked this pull request as ready for review February 20, 2026 19:02
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.

2 participants