Repository navigation
fix(mobile): simplify tool rich renderers - #5074
Merged
Merged
Conversation
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryIncremental review of the new accessibility commit (diff row Files Reviewed (10 files)
Previous Review Summary (commit fadc706)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit fadc706)Status: No Issues Found | Recommendation: Merge Files Reviewed (8 files)
Reviewed by kimi-k3 · Input: 70.9K · Output: 11.5K · Cached: 622.6K Review guidance: REVIEW.md from base branch |
jeanduplessis
approved these changes
Aug 6, 2026
iscekic
enabled auto-merge (squash)
August 6, 2026 08:50
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Users now see one image preview or file chip per tool part, without duplicated cached content. Markdown reads render the complete document directly in the scrollable detail sheet, without a duplicate full-reader action.
The mobile tool detail keeps full-screen image viewing, file sharing, selectable markdown, table access, and diff meaning. Error states and accessibility labels remain explicit.
The implementation follows the existing part-id image cache: render the first attachment, mount image viewers on demand, share markdown link interaction props, and add focused mounted and pure tests. The current main branch supplies the full markdown body design. No backend or dependency changes were made.
Human steps: none before or after merge.
Visual Changes: N/A.
E2E: bot-e2e — iOS verification covers image, direct markdown body plus table, and diff flows.