Skip to content

Unify app interfaces and fix Android editor and camera startup - #355

Merged
ymote merged 3 commits into
mainfrom
feat/octoscript-app-ux
Oct 7, 2026
Merged

ymote merged 3 commits into
mainfrom
feat/octoscript-app-ux

Conversation

@ymote

@ymote ymote commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

OctoScript system apps now share responsive, theme-aware surfaces, readable secondary text, larger actions and bounded desktop content. Calendar editing remains reachable above the Android keyboard, map tiles follow appearance changes, Glance chat has more readable spacing, and the native App Hub pin includes the matching navigation refresh. News follows an in-flight fetch without starting duplicate work.

A first Camera permission dialog previously changed an unprepared video into a false Playing state. The recorded runtime overlay preserves lifecycle state and waits for a single permission result before opening the camera. Video now preserves aspect ratio in Fill layouts using the resolved drawable size. Android provider QR scanning chooses supported Camera2 preview/YUV buffers, fits the visible preview proportionally, and retires callbacks before closing its surface so cancellation and reopening stay safe.

Validation:

  • 966 shell tests passed from phone/ with mobile-apps, including focused-editor resize and permission-dialog lifecycle regressions.
  • Desktop default/mobile and phone mobile checks, both shell graphs, runtime/source pin checks and 116 tooling tests passed.
  • Native UI journeys covered Unicode/multiline Calendar save and restart, unsaved theme changes, six-workspace retention, Mail draft editing, News saves, Photos with its mock provider, provider Add/Cancel and App Hub browsing.
  • Assigned Redmi Note 12: 8/8 curved Glance swipes, system Back, keyboard focus reveal, Save with IME open, saved detail review and warm-resume draft retention passed in a separate test package.

Camera follow-up validation:

  • macOS and Android release builds, three CameraPreview tests, the permission-dialog lifecycle regression, runtime patch integrity and 116 tooling tests passed. Existing compiler warnings remain.
  • Native Metal circle measurements changed from 304×462 to 462×462 and from 406×346 to 406×406 for a mirrored quarter turn.
  • Redmi Note 12: delayed first-use permission (~7 seconds), front/back switching, Photo/Video preview selection and warm resume exercised. The final APK passed three QR starts plus tap cancel, system Back and background interruption, without camera error, provider disconnect or crash.

Acceptance remains open: a clear-scene visual comparison on the phone is pending because the current camera view is dark/too close. QR decoding/provider import was not exercised. Android video recording is unimplemented. No 95/100 pass is claimed. Review is by the author, not an independent reviewer; raw pixel diagnostics are retained separately from the UX rubric. iOS/OHOS, assistive input, live email/model execution, map routing and video capture/playback remain unverified.

Companion: OctoSense-org/OctoSense-App-Hub#128 (merged; pinned f762267, including the host-sheet and key handling fixes already on main). Original device captures and private logs remain local; reproducible harnesses and bilingual instructions are in tools/app-ux/.

Runtime fixes: OctoSense-org/makepad#103. The recorded patches apply the same changes until the runtime pin is advanced.

Merge preparation: rebased onto main 1bc777a. The runtime lock now contains both the host authentication WebView overlay and the camera overlays; each patch applies cleanly and the computed runtime tree matches the lock. The three camera patch files and the tested Camera/AI providers app sources are unchanged by the rebase. Makepad #103 and App Hub #128 are merged. The Photos logic-test harness now starts after the host widget prelude; all 14 existing Photos memories tests pass locally. App Hub pins now target the merged revision f762267, preserving main's host fixes and published contract 1.5.0 while adding the UI refresh. Runtime graph, generated native-app configuration and shared interface checks pass. Full GitHub CI passed on head ee82715: desktop, home, apps, services and kernel-security. Main 1bc777a also has passing Desktop, Phone and Apps and services workflows.

Final CI: Desktop, Phone, Apps and services.

@ymote
ymote marked this pull request as ready for review October 7, 2026 08:06
@ymote
ymote force-pushed the feat/octoscript-app-ux branch from 8e6438f to 044fe1d Compare October 7, 2026 08:06
@ymote
ymote force-pushed the feat/octoscript-app-ux branch from d1549ff to ee82715 Compare October 7, 2026 08:46
@ymote
ymote merged commit d790b0a into main Oct 7, 2026
5 checks passed
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