Skip to content

fix(file-preview): select Markdown and text previews in place - #3197

Merged
stamp-build[bot] merged 1 commit into
mainfrom
agent/codex-android-devops/md-preview-inplace-select
Oct 20, 2026
Merged

stamp-build[bot] merged 1 commit into
mainfrom
agent/codex-android-devops/md-preview-inplace-select

Conversation

@stamp-build

@stamp-build stamp-build Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

View in Stamp

林舟 (Codex-Android-DevOPS) from slock-android opened this pull request through Stamp


What

Markdown and plain-text file previews become selectable in place: the body is wrapped in Compose SelectionContainer (androidx, post-#3192), so long-press + drag selects the rendered text and copy yields plain text (artin's request, replacing the #3193 "open Select Text page with Markdown source" approach).

  • Kept: overflow "Copy Markdown" / "Copy Text" (copy whole file).
  • Removed: overflow "Select Text", the long-press → dialog handler, SlockTextSelectionDialog.kt.
  • Changeset text updated.
  • task Persist diagnostics by cold-launch session #924's SelectionContainer ban targets the message Select Text dialog only; check_source_constraints.py passes.

Not verified yet (device-only)

  • Selection across Markdown blocks (paragraphs / lists / code blocks) — needs a device check.
  • Links in Markdown still open on tap inside the container.
  • No local JDK: CI is the first compile.

Why SelectionContainer is OK here but not for messages (task #924)

task #924 requires the message Select Text page to use the native selectable text view because SelectionContainer loses platform actions (Android PROCESS_TEXT, iOS Look Up / Translate). For the file preview, artin explicitly chose in-place selection of the rendered text over that (2026-10-20, "改,之前的规定也不重要"): the preview is a read-only document viewer where the need is "select a passage, copy plain text". Trade-off accepted: no translate / look-up in the preview selection menu. The message page keeps #924 unchanged.

1.14.2 counterpart (Kuikly): see release PR.

🤖 Generated with Claude Code

Wrap the Markdown and plain-text preview bodies in Compose's
SelectionContainer so long-press selects the rendered text directly and
copies plain text, instead of opening a separate Select Text page with
the Markdown source. "Copy Markdown" / "Copy Text" stays in the overflow
menu for copying the whole file; the Select Text menu entry and the
SlockTextSelectionDialog it opened are removed.

The task #924 SelectionContainer ban covers the message Select Text
dialog only and is unaffected.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@stamp-build
stamp-build Bot force-pushed the agent/codex-android-devops/md-preview-inplace-select branch from b428097 to 783a74d Compare October 20, 2026 03:34
@stamp-build
stamp-build Bot merged commit 6a18729 into main Oct 20, 2026
15 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.

0 participants