Skip to content

Tablet: Open Start chat in the detail pane - #463

Merged
thestinger merged 3 commits into
GrapheneOS:mainfrom
talanov:fix/tablet/new-chat-detail
Nov 2, 2026
Merged

thestinger merged 3 commits into
GrapheneOS:mainfrom
talanov:fix/tablet/new-chat-detail

Conversation

@talanov

@talanov talanov commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

fixes #380
tablet-convo-new

@RankoR
RankoR self-requested a review October 28, 2026 16:36
LaunchedEffect(Unit) {
// Focusing before the first layout crashes Compose's lookahead measure when the
// window turns two-pane and the list-detail scaffold is built around this screen
withFrameNanos {}

@RankoR RankoR Oct 28, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks hacky.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't disagree.

https://issuetracker.google.com/issues/567222742
On rotation, it tried to autofocus on a non-existing (now) element [rotation rebuilds everything?]

Here's an actual fix, I don't think it deserves a separate PR, but it's up to you.
8436281

@talanov
talanov force-pushed the fix/tablet/new-chat-detail branch from 491039e to 061cb86 Compare October 28, 2026 21:37
@talanov
talanov requested a review from RankoR October 28, 2026 21:38
@talanov
talanov force-pushed the fix/tablet/new-chat-detail branch from 061cb86 to 696393f Compare October 28, 2026 22:01

@RankoR RankoR left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for v19, also needs testing.

@AntiCot

AntiCot commented Nov 2, 2026

Copy link
Copy Markdown

Tested PR #463 on an Android 16 (API 36) tablet emulator.

Verified:

  • Start chat opens in the right detail pane.
  • The conversation list remains visible in the left pane.
  • Creating a new conversation using a randomly entered phone number.
  • Correct layout and navigation between the conversation list and the new chat.

No issues observed.

@thestinger
thestinger merged commit 2eb8ad6 into GrapheneOS:main Nov 2, 2026
7 checks passed
@talanov
talanov deleted the fix/tablet/new-chat-detail branch November 2, 2026 17:59
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.

Start chat takes over the entire large screen

4 participants