Skip to content

fix(mobile): PermissionSheet の表示安定性を改善 - #119

Open
dashi296 wants to merge 3 commits into
mainfrom
fix/mobile-permission-sheet-visibility
Open

fix(mobile): PermissionSheet の表示安定性を改善#119
dashi296 wants to merge 3 commits into
mainfrom
fix/mobile-permission-sheet-visibility

Conversation

@dashi296

@dashi296 dashi296 commented Apr 3, 2026

Copy link
Copy Markdown
Owner

概要

キーボード表示時に PermissionSheet が正しく位置調整されなかった問題を修正。

問題の原因

PermissionSheet 自身が position: absolutebottom: keyboardHeight を保持していたため、SafeAreaView の座標系と食い違いが生じ、キーボード表示中にシートの位置がずれる・隠れるケースがあった。

変更内容

PermissionSheet.tsx

  • position: absolute / left: 0 / right: 0 / bottom: 0 を除去し、純粋なコンテンツコンポーネントに変更
  • useKeyboardHeight 依存を除去
  • sheetPaddingBottom の計算をキーボード表示有無の分岐から 24 + bottomInset 固定に統一
  • 不要な zIndex: 10 を除去

TerminalScreen.tsx

  • permissionOverlay ラッパー View を追加し、position: absolutebottom: keyboardHeight をここで一元管理
  • pointerEvents="box-none" を設定し、背後の WebView / KeyboardToolbar へのタッチを妨げないように
  • zIndex: 10 に WebView / toolbar スタックの上に表示するための意図をコメントで明示

TerminalScreen.test.tsx

  • useKeyboardHeight のモック追加
  • キーボード表示中でも PermissionSheet が表示されることを確認するテストを追加
  • permissionOverlaybottom スタイル値が keyboardHeight と一致することをアサート
  • 既存テスト全件に createdAt フィールドを補完

テスト

全 125 テスト PASS

🤖 Generated with Claude Code

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.

1 participant