From 931b7fcff134590eb5fd0cc4934bf10f25e401ba Mon Sep 17 00:00:00 2001
From: thepra <thepra@pm.me>
Date: Tue, 6 Oct 2026 23:23:14 +0200
Subject: [PATCH] The globe opens full screen; a phone held sideways keeps its
 feed

On a phone the globe is a banner above the feed; a tap on it now opens it full screen over everything, with a close
button (Escape closes it too), the focus moving to that button and back, and the feed underneath kept as it was but out
of reach (inert). The camera refits the new shape. Held sideways, a phone (a window at most 30rem tall, the new short:
variant) has no room for the banner: the banner and its toggle give way to a button that opens the globe full screen.
isMobileMedia, which picks the first visit's default, agrees with Tailwind's md now (it was off by one) and counts a
phone held sideways. GlobePanel tells its state (data-globe-state). Phone flows: the globe opens full screen and
closes, and a sideways phone keeps a feed at least 180px tall with no sideways scroll.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LsXgEaXee4GCU1hwYgPJXw
---
 Components/Feed/FeedTabs.razor   |  9 +++++-
 Components/Feed/GlobePanel.razor | 55 ++++++++++++++++++++++++++++++--
 Pages/Index.razor                | 19 ++++++++---
 Resources/AllStrings.it.resx     |  3 ++
 Resources/AllStrings.resx        |  3 ++
 Styles/touch.css                 |  3 ++
 tests/e2e/Mobile/FlowTests.cs    | 43 +++++++++++++++++++++++++
 wwwroot/js/globe.js              |  9 ++++++
 wwwroot/main.js                  |  3 +-
 9 files changed, 139 insertions(+), 8 deletions(-)

diff --git a/Components/Feed/FeedTabs.razor b/Components/Feed/FeedTabs.razor
index afe0984..5684578 100644
--- a/Components/Feed/FeedTabs.razor
+++ b/Components/Feed/FeedTabs.razor
@@ -30,7 +30,12 @@
 		<NerdTip Heading="@Localizer["Feed"]" Rows="Stats" Placement="DTooltipPlacement.Bottom" Align="DTooltipAlign.End">
 			<i class="ion-md-stats opacity-50" aria-label="@Localizer["Feed"]"></i>
 		</NerdTip>
-		<DButton Size="DSize.Sm" class="lg:hidden" Active="Filter.ShowGlobe" OnClick="ToggleGlobe" title="@Localizer["Globe"]">
+		<DButton Size="DSize.Sm" class="lg:hidden max-lg:short:hidden" Active="Filter.ShowGlobe" OnClick="ToggleGlobe" title="@Localizer["Globe"]">
+			<i class="ion-md-planet text-base"></i>
+		</DButton>
+		@* a phone held sideways has no room for the banner: the globe opens full screen *@
+		<DButton Size="DSize.Sm" class="hidden max-lg:short:inline-flex" OnClick="() => OnGlobeFull.InvokeAsync()" title="@Localizer["Open the globe full screen"]"
+				 data-globe-full>
 			<i class="ion-md-planet text-base"></i>
 		</DButton>
 		<DButton Size="DSize.Sm" class="hidden md:inline-flex" Active="IsComposing" OnClick="() => OnCompose.InvokeAsync()" title="@Localizer["Post"]">
@@ -107,6 +112,8 @@
 	[Parameter] public IEnumerable<NerdRow> Stats { get; set; } = [];
 	[Parameter] public bool IsComposing { get; set; }
 	[Parameter] public EventCallback OnCompose { get; set; }
+	/// <summary>Opens the globe full screen (a phone held sideways, which has no room for its banner).</summary>
+	[Parameter] public EventCallback OnGlobeFull { get; set; }
 	/// <summary>Raised after Filter changed (tab, group or globe).</summary>
 	[Parameter] public EventCallback<FeedFilter> OnChanged { get; set; }
 
diff --git a/Components/Feed/GlobePanel.razor b/Components/Feed/GlobePanel.razor
index 4584a04..40358b7 100644
--- a/Components/Feed/GlobePanel.razor
+++ b/Components/Feed/GlobePanel.razor
@@ -6,8 +6,10 @@
    was written. The posts' places go to the module in one call whenever they change; the module calls SunChanged when
    the Sun's moment changes. The top corner shows that moment, the phase of the day and the season (its tooltip the
    Sun's numbers); the bottom corner the camera and the frame rate, its tooltip the rest of the globe's numbers. The
-   globe is isolated: its labels carry z-indexes by depth, which must not reach the corners' tooltips. *@
-<div class="relative w-full h-full">
+   globe is isolated: its labels carry z-indexes by depth, which must not reach the corners' tooltips. Below lg the
+   globe is a banner the page shows above the feed: a tap opens it full screen (Expanded, which the page lays out), and
+   its close button or Escape brings the feed back. *@
+<div class="relative w-full h-full" tabindex="-1" @onkeydown="OnKeyDown" data-globe-state="@State">
 	<div @ref="element" class="absolute inset-0 isolate" aria-label="@Localizer["Globe of the posts in view"]" role="img"></div>
 
 	@if (stats?.Sun is { } sun)
@@ -33,6 +35,21 @@
 		</div>
 	}
 
+	@if (Expanded)
+	{
+		<button @ref="closeButton" type="button" class="btn btn-circle btn-sm absolute right-3 top-[max(0.75rem,env(safe-area-inset-top))] z-20"
+				aria-label="@Localizer["Close"]" title="@Localizer["Close"]" @onclick="Close" data-globe-close>
+			<i class="ion-md-close" aria-hidden="true"></i>
+		</button>
+	}
+	else
+	{
+		<button @ref="expandButton" type="button" class="absolute inset-0 z-20 lg:hidden cursor-zoom-in" aria-label="@Localizer["Open the globe full screen"]"
+				title="@Localizer["Open the globe full screen"]" @onclick="Open" data-globe-expand>
+			<i class="ion-md-expand absolute right-3 bottom-3 text-lg rounded-full bg-base-100/70 p-1" aria-hidden="true"></i>
+		</button>
+	}
+
 	<div class="absolute left-2 bottom-2 z-10 flex items-center gap-2 text-[10px] font-mono opacity-70 pointer-events-auto">
 		<NerdTip Heading="@Localizer["Globe"]" Rows="StatsRows()" Placement="DTooltipPlacement.Top" Align="DTooltipAlign.Start">
 			<span class="badge badge-ghost badge-xs gap-1">
@@ -61,6 +78,30 @@
 	/// <summary>The id of the element that scrolls the feed's cards.</summary>
 	[Parameter, EditorRequired] public string ScrollContainerId { get; set; }
 	[Parameter] public List<FeedPost> Posts { get; set; } = [];
+	/// <summary>Open full screen (a phone's banner tapped); the page lays the globe out over everything.</summary>
+	[Parameter] public bool Expanded { get; set; }
+	[Parameter] public EventCallback<bool> ExpandedChanged { get; set; }
+
+	ElementReference closeButton;
+	ElementReference expandButton;
+	bool focusClose;
+	bool focusExpand;
+
+	string State => stats is null ? "loading" : stats.Error is { Length: > 0 } ? "error" : "live";
+
+	async Task Open()
+	{
+		focusClose = true;
+		await ExpandedChanged.InvokeAsync(true);
+	}
+
+	async Task Close()
+	{
+		focusExpand = true;
+		await ExpandedChanged.InvokeAsync(false);
+	}
+
+	Task OnKeyDown(KeyboardEventArgs e) => e.Key == "Escape" && Expanded ? Close() : Task.CompletedTask;
 
 	ElementReference element;
 	IJSObjectReference module;
@@ -78,6 +119,16 @@
 
 	protected override async Task OnAfterRenderAsync(bool firstRender)
 	{
+		if (focusClose && Expanded)
+		{
+			focusClose = false;
+			await closeButton.FocusAsync();
+		}
+		if (focusExpand && !Expanded)
+		{
+			focusExpand = false;
+			await expandButton.FocusAsync(preventScroll: true);
+		}
 		await base.OnAfterRenderAsync(firstRender);
 		try
 		{
diff --git a/Pages/Index.razor b/Pages/Index.razor
index 0fe53ba..173c7bc 100644
--- a/Pages/Index.razor
+++ b/Pages/Index.razor
@@ -13,11 +13,13 @@
    the stream tells is applied at once. *@
 <div class="flex flex-col lg:grid lg:grid-cols-[minmax(0,1fr)_minmax(320px,42%)] gap-3 md:gap-4 w-full h-full">
 
-	<aside class="@(Filter.ShowGlobe ? "block" : "hidden") lg:block lg:order-last relative h-56 sm:h-64 lg:h-full flex-none min-h-0 rounded-xl neo-raised neo-xs overflow-hidden">
-		<GlobePanel @ref="globe" ScrollContainerId="@ScrollId" Posts="posts" />
+	@* full screen over everything once a phone's banner is tapped; the feed underneath stays as it was, out of reach *@
+	<aside class="@GlobeClass" role="@(globeExpanded ? "dialog" : null)" aria-modal="@(globeExpanded ? "true" : null)"
+		   aria-label="@(globeExpanded ? Localizer["Globe"] : null)">
+		<GlobePanel @ref="globe" ScrollContainerId="@ScrollId" Posts="posts" @bind-Expanded="globeExpanded" />
 	</aside>
 
-	<section class="flex flex-col gap-3 min-h-0 flex-1 lg:h-full">
+	<section class="flex flex-col gap-3 min-h-0 flex-1 lg:h-full" inert="@globeExpanded">
 		@if (!isComposing)
 		{
 			@* on a phone a new post starts from a button under the thumb; from md, from the feed's tabs *@
@@ -27,7 +29,7 @@
 				<i class="ion-md-create text-xl" aria-hidden="true"></i>
 			</DButton>
 		}
-		<FeedTabs Filter="Filter" Groups="groups" Lists="lists" UnreadNotifications="unreadNotifications" UnreadConversations="unreadConversations" Stats="FeedStats()" IsComposing="isComposing" OnCompose="ToggleCompose"
+		<FeedTabs Filter="Filter" Groups="groups" Lists="lists" UnreadNotifications="unreadNotifications" UnreadConversations="unreadConversations" Stats="FeedStats()" IsComposing="isComposing" OnCompose="ToggleCompose" OnGlobeFull="() => globeExpanded = true"
 				  OnChanged="ChangeFeed" />
 
 		<div class="relative flex-1 min-h-0 neo-inset neo-xs rounded-xl">
@@ -141,6 +143,15 @@
 	string loadError;
 	bool isLoadingMore;
 	bool isComposing;
+	bool globeExpanded;
+
+	// a phone's banner (none when the phone is held sideways: the feed would have no room), the column from lg, or
+	// full screen
+	string GlobeClass => globeExpanded
+		? "fixed inset-0 z-[70] block h-dvh w-full overflow-hidden bg-base-100"
+		: Filter.ShowGlobe
+			? "block max-lg:short:hidden lg:block lg:order-last relative h-56 sm:h-64 lg:h-full flex-none min-h-0 rounded-xl neo-raised neo-xs overflow-hidden"
+			: "hidden lg:block lg:order-last relative h-56 sm:h-64 lg:h-full flex-none min-h-0 rounded-xl neo-raised neo-xs overflow-hidden";
 	(double Latitude, double Longitude)? position;
 	string positionError;
 	string loadedPersonaId;
diff --git a/Resources/AllStrings.it.resx b/Resources/AllStrings.it.resx
index e6b6410..9e6c511 100644
--- a/Resources/AllStrings.it.resx
+++ b/Resources/AllStrings.it.resx
@@ -1823,4 +1823,7 @@
   <data name="New post" xml:space="preserve">
     <value>Nuovo post</value>
   </data>
+  <data name="Open the globe full screen" xml:space="preserve">
+    <value>Apri il globo a schermo intero</value>
+  </data>
 </root>
diff --git a/Resources/AllStrings.resx b/Resources/AllStrings.resx
index 70a33f7..b4b8258 100644
--- a/Resources/AllStrings.resx
+++ b/Resources/AllStrings.resx
@@ -1823,4 +1823,7 @@
   <data name="New post" xml:space="preserve">
     <value>New post</value>
   </data>
+  <data name="Open the globe full screen" xml:space="preserve">
+    <value>Open the globe full screen</value>
+  </data>
 </root>
diff --git a/Styles/touch.css b/Styles/touch.css
index 1b7f305..9300f2f 100644
--- a/Styles/touch.css
+++ b/Styles/touch.css
@@ -28,6 +28,9 @@
 	}
 }
 
+/* A short window: a phone held sideways. The globe's banner would leave the feed no room there. */
+@custom-variant short (@media (max-height: 30rem));
+
 @layer utilities {
 	/* At every width: a checkbox's, radio's or toggle's label wraps (daisyUI's .label never does, and a long one ran past
 	   a phone's edge), and the control keeps its size beside it. */
diff --git a/tests/e2e/Mobile/FlowTests.cs b/tests/e2e/Mobile/FlowTests.cs
index 107fea2..2491520 100644
--- a/tests/e2e/Mobile/FlowTests.cs
+++ b/tests/e2e/Mobile/FlowTests.cs
@@ -182,6 +182,49 @@ namespace decePubClient.E2E.Mobile
 			});
 		}
 
+		[Fact]
+		public async Task The_globe_opens_full_screen_from_its_banner_and_closes()
+		{
+			await using var session = await SignIn.As(browser, Persona(), Phone);
+			await Flow(session, "globe", async page =>
+			{
+				await page.Locator("button[title='Globe']").TapAsync();
+				var expand = page.Locator("[data-globe-expand]");
+				await Assertions.Expect(expand).ToBeVisibleAsync();
+				await expand.TapAsync();
+				var full = page.Locator("aside[role=dialog]");
+				await Assertions.Expect(full).ToBeVisibleAsync();
+				var box = (await full.BoundingBoxAsync())!;
+				var size = page.ViewportSize!;
+				Assert.True(Math.Abs(box.Width - size.Width) < 2 && Math.Abs(box.Height - size.Height) < 2,
+					$"the globe is {box.Width}x{box.Height}, not the screen's {size.Width}x{size.Height}");
+				await AssertWhole(page, "aside[role=dialog]");
+				await page.Locator("[data-globe-close]").TapAsync();
+				await Assertions.Expect(full).ToHaveCountAsync(0);
+				await Assertions.Expect(expand).ToBeVisibleAsync();
+			});
+		}
+
+		// a phone held sideways keeps its feed: no banner squeezing it, the globe full screen from the tabs instead
+		[Fact]
+		public async Task A_phone_held_sideways_keeps_its_feed()
+		{
+			await using var session = await SignIn.As(browser, Persona(), Phone.Pixel7Landscape);
+			var page = session.Page;
+			await Results.Cell("client.mobile.flow.landscape", "privapub", Phone.Pixel7Landscape.Key, async () =>
+			{
+				Assert.False(await page.Locator("[data-globe-expand]").IsVisibleAsync(), "the globe's banner squeezes the feed");
+				var feedHeight = await page.EvaluateAsync<double>("() => document.getElementById('feed-scroll').clientHeight");
+				Assert.True(feedHeight >= 180, $"the feed is {feedHeight}px tall");
+				Assert.True(await page.EvaluateAsync<bool>("() => document.scrollingElement.scrollWidth <= innerWidth + 1"), "the page scrolls sideways");
+				await page.Locator("[data-globe-full]").TapAsync();
+				await Assertions.Expect(page.Locator("aside[role=dialog]")).ToBeVisibleAsync();
+				await page.Locator("[data-globe-close]").TapAsync();
+				await Assertions.Expect(page.Locator("aside[role=dialog]")).ToHaveCountAsync(0);
+				await session.AssertGuards();
+			}, page);
+		}
+
 		static async Task<JsonNode> Latest(Pasture.Persona persona) =>
 			(await PersonaApi.Call(persona, $"/api/v1/accounts/{persona.LocalId}/statuses?limit=1"))!.AsArray().FirstOrDefault();
 	}
diff --git a/wwwroot/js/globe.js b/wwwroot/js/globe.js
index 056d36b..16a1f00 100644
--- a/wwwroot/js/globe.js
+++ b/wwwroot/js/globe.js
@@ -198,6 +198,15 @@ function resize() {
 	state.globe.width(width).height(height)
 	pause(document.visibilityState !== "visible")
 	wake()
+	// a new shape (the phone's banner opened full screen, or closed again) wants the camera at another height
+	const shape = narrowness()
+	if (Math.abs(shape - (state.shape ?? shape)) > 0.05) {
+		if (state.view)
+			scheduleView()
+		else
+			state.globe.pointOfView({ ...state.globe.pointOfView(), altitude: 2.6 * shape }, reducedMotion.matches ? 0 : 600)
+	}
+	state.shape = shape
 }
 
 async function build(width, height) {
diff --git a/wwwroot/main.js b/wwwroot/main.js
index d729673..ae077be 100644
--- a/wwwroot/main.js
+++ b/wwwroot/main.js
@@ -123,9 +123,10 @@ window.clearAppBadge = () => {
 		window.logFromJs(e.message, "clearAppBadge")
 	}
 }
+// a phone: narrower than Tailwind's md, or held sideways (as short as the short: variant)
 window.isMobileMedia = () => {
 	try {
-		return ((window.innerWidth > 0) ? window.innerWidth : screen.width) <= 768
+		return window.matchMedia("(max-width: 767.98px), (max-height: 30rem)").matches
 	} catch (e) {
 		console.error(e)
 		window.logFromJs(e.message, "isMobileMedia")