Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions apps/mobile/src/features/workspace/WorkspaceRootScreen.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ function WorkspaceRootContent() {
const nativeMetrics = useNativeLayoutMetrics();
const systemTabBar = useSystemWorkspaceTabBar();
const sidebarTabs = useSidebarWorkspaceTabs();
const sidebarPicker = sidebarTabs === "picker";
const sidebarNavigation =
primaryColumn !== null &&
(layout.usesSplitView ||
Expand Down Expand Up @@ -98,8 +99,8 @@ function WorkspaceRootContent() {
return (
<WorkspaceTabsContext value={context}>
<View
className={sidebarTabs ? "flex-1 bg-drawer" : "flex-1 bg-screen"}
style={sidebarTabs ? { paddingTop: insets.top } : undefined}
className={sidebarPicker ? "flex-1 bg-drawer" : "flex-1 bg-screen"}
style={sidebarPicker ? { paddingTop: insets.top } : undefined}
>
{Platform.OS === "ios" ? (
<NativeStackScreenOptions
Expand Down Expand Up @@ -136,8 +137,8 @@ function WorkspaceRootContent() {
]}
/>
)}
{sidebarTabs ? <WorkspaceSidebarTabs /> : null}
<SafeAreaInsetsContext value={sidebarTabs ? { ...insets, top: 0 } : insets}>
{sidebarPicker ? <WorkspaceSidebarTabs /> : null}
<SafeAreaInsetsContext value={sidebarPicker ? { ...insets, top: 0 } : insets}>
{state.selectedTab === "threads" ? <HomeRouteScreen /> : <PullRequestsScreen />}
</SafeAreaInsetsContext>
<WorkspaceToolbar
Expand Down
13 changes: 4 additions & 9 deletions apps/mobile/src/features/workspace/WorkspaceSidebarTabs.ios.tsx
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
import { StackActions, useNavigation } from "@react-navigation/native";
import { Keyboard, View } from "react-native";
import { View } from "react-native";
import { SegmentedControl } from "../../components/ui/SegmentedControl";
import { useWorkspaceTabs } from "./WorkspaceTabsContext";
import type { WorkspaceTab } from "./workspace-tabs-state";
import { useSelectWorkspaceTab } from "./useSelectWorkspaceTab";

export function WorkspaceSidebarTabs() {
const workspace = useWorkspaceTabs()!;
const navigation = useNavigation();
const selectTab = useSelectWorkspaceTab();
return (
<View className="px-4 pb-3 pt-2">
<SegmentedControl
Expand All @@ -16,12 +16,7 @@ export function WorkspaceSidebarTabs() {
{ value: "pullRequests", label: "Pull requests" },
]}
selected={workspace.state.selectedTab}
onSelect={(tab: WorkspaceTab) => {
if (tab === workspace.state.selectedTab) return;
Keyboard.dismiss();
if (navigation.getState()?.index) navigation.dispatch(StackActions.popToTop());
workspace.dispatch({ type: "select", tab });
}}
onSelect={(tab: WorkspaceTab) => selectTab(tab)}
/>
</View>
);
Expand Down
30 changes: 29 additions & 1 deletion apps/mobile/src/features/workspace/WorkspaceToolbar.ios.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,12 @@ import { NativeWorkspaceTabActivityContext } from "../../native/v5-workspace-con
import { useHardwareKeyboardCommand } from "../keyboard/hardwareKeyboardCommands";
import { useAdaptiveWorkspaceLayout } from "../layout/AdaptiveWorkspaceLayout";
import { useWorkspaceTabs } from "./WorkspaceTabsContext";
import { useSidebarWorkspaceTabs } from "./useSystemWorkspaceTabBar";
import { useSelectWorkspaceTab } from "./useSelectWorkspaceTab";
import { useMobileNavigationTheme } from "../../lib/useMobileNavigationTheme";

// Tabler's git-pull-request geometry, shared with the UIKit tab bar.
const PULL_REQUEST_ICON = require("../../../assets/icons/pull-request.png");

export function WorkspaceToolbar(props: {
readonly onNewTask: () => void;
Expand All @@ -19,8 +25,12 @@ export function WorkspaceToolbar(props: {
const workspace = useWorkspaceTabs()!;
const navigation = useNavigation();
const metrics = useNativeLayoutMetrics();
const sidebarTabs = useSidebarWorkspaceTabs();
const selectTab = useSelectWorkspaceTab();
const selectedTint = useMobileNavigationTheme().colors.primary;
const nativeToolbarCompose =
Platform.OS === "ios" && (Platform.isPad || (metrics?.verticalBarEdge ?? "none") !== "none");
Platform.OS === "ios" &&
(Platform.isPad || sidebarTabs !== null || (metrics?.verticalBarEdge ?? "none") !== "none");
const { isTabActive } = workspace;
const tabActive = use(NativeWorkspaceTabActivityContext)?.active ?? true;
const { panes, togglePrimarySidebar } = useAdaptiveWorkspaceLayout();
Expand Down Expand Up @@ -122,6 +132,24 @@ export function WorkspaceToolbar(props: {
)}
</Toolbar>
<Toolbar placement="bottom">
{sidebarTabs === "toolbar" ? (
<Toolbar.Button
identifier="workspace-tab-threads"
accessibilityLabel="Threads"
icon={selectedTab === "threads" ? "text.bubble.fill" : "text.bubble"}
tintColor={selectedTab === "threads" ? selectedTint : undefined}
onPress={() => selectTab("threads")}
/>
) : null}
{sidebarTabs === "toolbar" ? (
<Toolbar.Button
identifier="workspace-tab-pull-requests"
accessibilityLabel="Pull requests"
icon={{ type: "image", source: PULL_REQUEST_ICON, tinted: true }}
tintColor={selectedTab === "pullRequests" ? selectedTint : undefined}
onPress={() => selectTab("pullRequests")}
/>
) : null}
{nativeToolbarCompose ? <Toolbar.Spacer flexible /> : null}
{nativeToolbarCompose ? (
<Toolbar.Button
Expand Down
22 changes: 22 additions & 0 deletions apps/mobile/src/features/workspace/useSelectWorkspaceTab.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
import { StackActions, useNavigation } from "@react-navigation/native";
import { useCallback } from "react";
import { Keyboard } from "react-native";
import { useWorkspaceTabs } from "./WorkspaceTabsContext";
import type { WorkspaceTab } from "./workspace-tabs-state";

/** Switch workspace tabs from inside the sidebar, closing any open detail first. */
export function useSelectWorkspaceTab() {
const workspace = useWorkspaceTabs()!;
const navigation = useNavigation();
const selectedTab = workspace.state.selectedTab;
const { dispatch } = workspace;
return useCallback(
(tab: WorkspaceTab) => {
if (tab === selectedTab) return;
Keyboard.dismiss();
if (navigation.getState()?.index) navigation.dispatch(StackActions.popToTop());
dispatch({ type: "select", tab });
},
[dispatch, navigation, selectedTab],
);
}
13 changes: 10 additions & 3 deletions apps/mobile/src/features/workspace/useSystemWorkspaceTabBar.ts
Original file line number Diff line number Diff line change
@@ -1,12 +1,19 @@
import { Platform } from "react-native";
import { useAdaptiveWorkspaceLayout } from "../layout/AdaptiveWorkspaceLayout";
import { useNativeLayoutMetrics } from "../../native/native-layout-metrics";

export function useSidebarWorkspaceTabs() {
/** Where an iOS split window shows the workspace tabs instead of UIKit's tab bar. */
export function useSidebarWorkspaceTabs(): "picker" | "toolbar" | null {
const { layout } = useAdaptiveWorkspaceLayout();
return Platform.OS === "ios" && Platform.isPad && layout.usesSplitView;
const metrics = useNativeLayoutMetrics();
if (Platform.OS !== "ios" || !layout.usesSplitView) return null;
if (Platform.isPad) return "picker";
// The Duo's side rail carries UIKit's tabs. Unfolded in portrait it has no
// rail, so the tabs sit in the thread list's own toolbar.
return (metrics?.verticalBarEdge ?? "none") === "none" ? "toolbar" : null;
}

export function useSystemWorkspaceTabBar() {
const sidebarTabs = useSidebarWorkspaceTabs();
return Platform.OS === "ios" && !sidebarTabs;
return Platform.OS === "ios" && sidebarTabs === null;
}
5 changes: 4 additions & 1 deletion apps/mobile/src/lib/layout.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,12 @@ export function deriveLayout(input: {
const usableWidth = width - (metrics?.safeArea.left ?? 0) - (metrics?.safeArea.right ?? 0);
// UIKit's size class supports the shorter Duo inner display. Geometry remains
// a floor so a narrow multitasking window cannot squeeze both columns.
// Unfolded in portrait, the inner display reports neither a side bar nor a
// fold, only its hinge.
const hasDuoGeometry =
metrics &&
(metrics.verticalBarEdge !== "none" ||
(metrics.hasHinge === true ||
metrics.verticalBarEdge !== "none" ||
metrics.reservedRegions.some((region) => region.kind === "division"));
const wideEnoughForSplit = hasDuoGeometry
? metrics.horizontalSizeClass === "regular" &&
Expand Down
37 changes: 37 additions & 0 deletions apps/mobile/src/lib/reserved-regions.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,43 @@ describe("Duo workspace sizing", () => {
).toBe(false);
});

it("shows both columns on the unfolded inner display in portrait", () => {
// Reported by the iOS 27.1 Duo simulator: no side bar or fold, only the hinge.
const portrait: NativeLayoutMetrics = {
width: 669,
height: 951,
hasHinge: true,
horizontalSizeClass: "regular",
verticalBarEdge: "none",
safeArea: { top: 82, bottom: 34, left: 0, right: 0 },
reservedRegions: [{ kind: "occlusion", x: 535, y: 0, width: 134, height: 82 }],
};
expect(deriveLayout({ ...portrait, nativeMetrics: portrait }).usesSplitView).toBe(true);
// Without the hinge, the same window is an ordinary narrow iPad-style window.
const withoutHinge = { ...portrait, hasHinge: false };
expect(deriveLayout({ ...withoutHinge, nativeMetrics: withoutHinge }).usesSplitView).toBe(
false,
);
});

it("keeps the folded front screen compact in both orientations", () => {
for (const [width, height] of [
[466, 678],
[678, 466],
]) {
const folded: NativeLayoutMetrics = {
width: width!,
height: height!,
hasHinge: true,
horizontalSizeClass: "compact",
verticalBarEdge: "right",
safeArea: { top: 0, bottom: 34, left: 0, right: 84 },
reservedRegions: [],
};
expect(deriveLayout({ ...folded, nativeMetrics: folded }).usesSplitView).toBe(false);
}
});

it("requires enough safe width for both panes in a multitasking window", () => {
const metrics = { ...innerDisplay, width: 660 };
expect(deriveLayout({ ...metrics, nativeMetrics: metrics }).usesSplitView).toBe(false);
Expand Down
Loading