Skip to content

[RNE Rewrite] fix(example): add bottom safe area inset to classification screen#1299

Merged
barhanc merged 1 commit into
rne-rewritefrom
@bh/fix-classification-safe-area
Jul 3, 2026
Merged

[RNE Rewrite] fix(example): add bottom safe area inset to classification screen#1299
barhanc merged 1 commit into
rne-rewritefrom
@bh/fix-classification-safe-area

Conversation

@barhanc

@barhanc barhanc commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description

Follow-up PR to two problems raised in review of #1296

It

  • Fixes the classification example screen to not overlap with Android navigation buttons.
  • Leaves the default YOLO threshold as 0.25 since it is the official recommended default value (see: ultralytics docs)

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

  • Run the computer-vision app on Android and confirm that there is no overlap with navigation buttons.

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@barhanc barhanc requested review from Copilot and msluszniak and removed request for Copilot July 3, 2026 14:12
@barhanc barhanc self-assigned this Jul 3, 2026
@barhanc barhanc added the bug fix PRs that are fixing bugs label Jul 3, 2026
@barhanc barhanc merged commit 5b4eef0 into rne-rewrite Jul 3, 2026
3 checks passed
@barhanc barhanc deleted the @bh/fix-classification-safe-area branch July 3, 2026 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug fix PRs that are fixing bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants