diff --git a/apps/mobile/src/app/(app)/pr-review/[owner]/[repo]/[number]/_layout.tsx b/apps/mobile/src/app/(app)/pr-review/[owner]/[repo]/[number]/_layout.tsx index aa75b15b70..3d8b4cc16d 100644 --- a/apps/mobile/src/app/(app)/pr-review/[owner]/[repo]/[number]/_layout.tsx +++ b/apps/mobile/src/app/(app)/pr-review/[owner]/[repo]/[number]/_layout.tsx @@ -51,6 +51,7 @@ export default function PrReviewNumberLayout() { const sheetOptions = { presentation: 'formSheet' as const, sheetAllowedDetents: [0.5, fullSheetDetent] as [number, number], + sheetInitialDetentIndex: 'last' as const, sheetGrabberVisible: true, headerShown: false, }; diff --git a/apps/mobile/src/components/pr-review/merge/pr-merge-sheet-parts.tsx b/apps/mobile/src/components/pr-review/merge/pr-merge-sheet-parts.tsx index f2ec085882..b6073aad6f 100644 --- a/apps/mobile/src/components/pr-review/merge/pr-merge-sheet-parts.tsx +++ b/apps/mobile/src/components/pr-review/merge/pr-merge-sheet-parts.tsx @@ -43,11 +43,11 @@ function MethodPicker({ onChange: (next: AllowedMergeMethod) => void; }>) { return ( - - + + Method - + {methodOptions.map(option => { const active = method === option.value; // Long labels stay readable via accessibilityLabel; chip shows short text. @@ -62,7 +62,7 @@ function MethodPicker({ {...radioItemA11y({ label: option.label, checked: active, disabled: isDisabled })} accessibilityHint={PR_MERGE_DESCRIPTIONS[option.value]} className={cn( - 'min-h-8 items-center justify-center rounded-full border px-2.5 py-1 active:opacity-70', + 'min-h-11 items-center justify-center rounded-full border px-4 py-2 active:opacity-70', active && 'border-primary bg-primary', !active && isDisabled && 'border-hair-soft bg-secondary', !active && !isDisabled && 'border-border bg-secondary' @@ -70,7 +70,7 @@ function MethodPicker({ > ) { const colors = useThemeColors(); return ( - - Commit title + + Commit title - Commit message + + Commit message + Delete branch - + {noMethodsAllowed ? ( - + ) : ( <>