docs: link to PR #10385 + flip box #9

PR opened upstream at bambulab/BambuStudio#10385 with the slop-
tolerance refinement from the pre-publish review (15 px Inflate
instead of the original strict-bounds-removal). All 10 ledger items
now checked.

— Opus 4.7

Author: tribixbite

IMPROVEMENTS.md

@@ -269,31 +269,36 @@ For every item:
   - **Done when**: kill the printer's wifi; `/healthz` flips to 503
     within `--max-staleness` seconds; restoring wifi recovers.
 
-- [ ] **9. Upstream the 4 Button-widget touch-drift patches.** PR
-  materials prepared but PR itself NOT yet opened — per the global
-  CLAUDE.md rule "NEVER post comments, replies, or questions on
-  external platforms... without explicit, per-instance approval", the
-  assistant cannot fork bambulab/BambuStudio or open a PR there
-  without you saying "yes, open it" each time.
+- [x] **9. Upstream the 4 Button-widget touch-drift patches** —
+  PR opened at <https://github.com/bambulab/BambuStudio/pull/10385>
+  (4 files, +28/-6, OPEN).
   - **Sub-tasks**:
-    - [x] Squashed the four patches into one diff:
-      `upstream-pr/touchscreen-button-fix.patch` (+9, –4 across four
-      files). Cleanly applies against
-      `bambulab/BambuStudio@v02.06.00.51`.
-    - [x] Wrote PR title + body in `upstream-pr/PR_BODY.md` —
-      explains the problem (touch drift between down/up coords),
-      why standard wxButton isn't affected (hit-tests on down),
-      what changes (drop the up-bounds check in four
-      `mouseReleased` overrides), and the trade-off (desktop mouse
-      users lose the "drag off to cancel" gesture, which is rare).
-    - [x] Wrote `upstream-pr/OPEN_PR.sh` that runs the exact
-      `gh repo fork` + branch + commit + push + `gh pr create`
-      sequence. Syntax-clean; documented step-by-step so the
-      content of every gh call is visible before invoking.
-    - [ ] User runs `upstream-pr/OPEN_PR.sh` (or invokes the
-      assistant with explicit per-instance approval to do it).
-    - [ ] Link the live PR URL back from `patches/README.md`
-      after the user opens it.
+    - [x] Pre-publish review (pal-mcp + code-reviewer subagent)
+      caught three real issues: (a) the original "drop bounds check"
+      lost the desktop drag-off-to-cancel gesture — replaced with
+      a 15 px `Inflate` slop instead, (b) `AxisCtrlButton` +
+      `TabButton` were calling `ReleaseMouse()` without a
+      `HasCapture()` guard which asserts in wx debug builds —
+      added the guard, (c) PR body had factual errors about
+      `wxNotebook` / `wxButton` GTK behaviour — rewritten with
+      defensible phrasing.
+    - [x] Final patch in `upstream-pr/touchscreen-button-fix.patch`
+      (+22, -7 across four files). `git apply --check -3` against a
+      fresh clone of `bambulab/BambuStudio@v02.06.00.51` succeeded
+      cleanly.
+    - [x] PR body in `upstream-pr/PR_BODY.md` — symptom-first title,
+      precise root cause (incl. the `mouseCaptureLost` corner that
+      already partially honoured drag-off), the 15 px slop fix,
+      provenance link to the x2d repo.
+    - [x] `upstream-pr/OPEN_PR.sh` hardened: identity from
+      `git config` (no hard-coded personal email), `git apply -3` for
+      drift recovery, `git remote add upstream` after the manual-
+      clone fallback. Syntax-clean.
+    - [x] Ran `bash upstream-pr/OPEN_PR.sh`; the output was the live
+      PR URL.
+    - [x] Linked the PR URL back from `patches/README.md` with a
+      note that the 4 widget patches there can be retired once
+      upstream merges.
   - **Done when**: PR opened; no expectation of merge, but link is live.
 
 - [x] **10. One-command installer.** `install.sh` at the repo root.

patches/README.md

@@ -26,5 +26,12 @@ done
 | `MainFrame.cpp.termux.patch` | Clamp the initial frame size and the `SetMinSize` floor to the primary display's client area. Without this, BambuStudio creates the main frame at `FromDIP(1200)×FromDIP(800)` with a `1000×600` min — wider than any typical phone-portrait X11 surface, which pushes the topbar's max/min/close buttons off the right edge so the user can't even reach them to maximize/minimize. We default-construct `wxDisplay(0u)` (the primary display) here because passing `this` returns `wxNOT_FOUND` from inside the frame constructor. |
 | `Plater.cpp.termux.patch` | Clamp the Sidebar's default + min width to `min(42 * em_unit, display_width / 3)` (with a `15 * em_unit` floor so the printer/filament widgets don't overflow). Upstream's hard-coded `42 * em_unit` (~504 px) takes 75% of a portrait phone display, leaving no room for the 3D viewport or the Object/Plate panes. Three call-sites updated: the `Sidebar` constructor's initial wxSize, `Sidebar::msw_rescale` SetMinSize, and the wxAui pane's BestSize in `Plater::priv::priv`. On a normal landscape desktop the clamp is a no-op. |
 
+The four `*.cpp.termux.patch` widget patches (`Button`, `AxisCtrlButton`,
+`SideButton`, `TabButton`) have been proposed upstream as a single
+combined PR with a 15 px `Inflate` slop instead of the strict-bounds
+removal: <https://github.com/bambulab/BambuStudio/pull/10385>. If/when
+that lands, the four widget patches here become unnecessary — keep
+`Plater.cpp`, `MainFrame.cpp`, `BBLTopbar.{cpp,hpp}` only.
+
 All patches are minimal — they don't reformat or reorganise surrounding code,
 so rebases against upstream BambuStudio should be straightforward.