Skip to content

Pin gpui-mobile with the night-mode and emoji review fixes - #22

Merged
Bombatomica64 merged 1 commit into
mainfrom
pin-review-fixes
Oct 9, 2026
Merged

Bombatomica64 merged 1 commit into
mainfrom
pin-review-fixes

Conversation

@Bombatomica64

Copy link
Copy Markdown
Owner

Pins fork main 3d74121, which merges the revised android-host-night-mode (c62c318) and android-host-emoji-font (77126f2). Those are what longbridge/gpui-mobile#30 and #29 will carry after the adversarial review:

  • Night mode: read from the application context's Resources, kept once per process. Before, it came from whichever Activity registered last, which could be a finished one. configuration_changed() now syncs every host's window.
  • Emoji: the application's AssetManager is kept for the life of the process, with no hand-written unsafe impl Send. It is lent out through a crate-private with_asset_manager, replacing the public asset_manager() that could return a dangling pointer. A failed JNI call no longer leaves an exception pending.

This APK is for re-testing both on the phone before force-pushing the upstream branches.

🤖 Generated with Claude Code

Fork main 3d74121 merges the revised android-host-night-mode (c62c318) and
android-host-emoji-font (77126f2), which longbridge/gpui-mobile#30 and #29
will carry: night mode is read from the application context, and the
emoji font's AssetManager is kept for the life of the process.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Bombatomica64
Bombatomica64 merged commit 6f38cd4 into main Oct 9, 2026
1 check 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