Repository navigation
Conversation
|
@besmpl, this has sat here without a review from us for almost three months. Sorry about that — the IME contract it depends on was still open on our side, and the PR waited on it instead of us telling you where it stood. The behaviour you built is what we want: preedit kept apart from committed text, marked ranges and underline, the candidate cursor rect, UTF-8-safe delete-surrounding, and password and sensitive fields kept out of composition. The contract is settled now and released as gpucontext v0.32.0 (
gogpu's platform IME backends aren't in yet, so the widget side will be built against the contract with a test double and wired end to end when they land. Rather than make you wait any longer, we'll do this rework ourselves on your branch. Your authorship stays, and the changelog credits you. If you'd rather pick it back up yourself, say so and we'll hold. |
Summary
Dependencies
This PR depends on:
The branch contains no committed
replacedirectives and intentionally remains draft until both dependencies are merged and released. After releases are available, updatego.mod, rungo mod tidy, rerun the matrix below, and mark this PR ready.Verification
Dependency validation used temporary modfile replacements only; none are committed.
go test -count=1 ./...: PASSgo build ./...and focusedgo vet: PASSgofmtandgit diff --check: PASSThe full app race suite still reproduces the pre-existing animation-pumper mock
RequestRedrawrace outside this diff. Live native compositor/locale testing requires target hosts.Refs gogpu/gogpu#331. Depends on gogpu/gpucontext#30 and gogpu/gogpu#462.