Skip to content

fix(mobile): keep ordered lists inside user bubbles - #6154

Open
none23 wants to merge 2 commits into
pingdotgg:mainfrom
none23:t3code/fix-android-ordered-lists
Open

fix(mobile): keep ordered lists inside user bubbles#6154
none23 wants to merge 2 commits into
pingdotgg:mainfrom
none23:t3code/fix-android-ordered-lists

Conversation

@none23

@none23 none23 commented Aug 11, 2026

Copy link
Copy Markdown

What Changed

Extended fixed-width Markdown detection to recognize ordered-list markers in user messages.
Ordered lists now use the same definite-width Android bubble layout already used for fenced code blocks and GFM tables.

Why

Android can incorrectly measure list-item flex rows inside shrink-to-content user bubbles, causing ordered-list content to render outside the bubble and overlap subsequent messages.

A small regular-expression heuristic keeps the fix focused and avoids changing the shared list renderer. It trades some precision for simplicity, favoring harmless wider bubbles over false negatives that leave messages unreadable.

Fixes #6109.

UI Changes

Before After
Ordered list escaping an Android user-message bubble image

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes - (not applicable)

Note

Low Risk
Narrow mobile layout heuristic and tests only; no auth, data, or shared list renderer changes.

Overview
User messages that look like ordered lists now trigger the same pinned-width bubble layout already used for fenced code and GFM tables on Android.

hasWideMarkdownBlock gains an ORDERED_LIST_ITEM regex (up to three spaces, digits, . or ), then space/tab or EOL). Indented four-space 1. lines stay excluded so nested continuations are not misclassified. The heuristic prefers false positives (slightly wider bubbles) over missing real lists that would overlap other messages.

Unit tests cover multi-item lists, 3) style markers, and the indented false-negative case.

Reviewed by Cursor Bugbot for commit 65bf3bc. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Fix ordered lists rendering inside user bubbles on mobile

Extends hasWideMarkdownBlock in wideMarkdownBlocks.ts to detect ordered list markers (e.g. 1. item or 3) item) in addition to fenced code blocks and GFM tables. This causes messages containing ordered lists to use the wide layout path, keeping list formatting intact inside user chat bubbles.

Macroscope summarized 65bf3bc.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7a54802f-9fbf-4893-8b95-4abb9e6790c2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Aug 11, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 5807b43

Straightforward mobile UI bug fix adding ordered list detection to existing markdown block sizing logic. Changes are minimal, well-tested, and documented. Risk is cosmetic only (possible wider bubbles for list-like text).

You can customize Macroscope's approvability policy. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Ordered lists escape user-message bubbles on Android

1 participant