Skip to content

01a09c2f - Filter the inbox by Direct, Contact, or Damus - #178

Merged
TaprootFreak merged 17 commits into
developfrom
01a09c2f-inbox-from-me
Sep 14, 2026
Merged

TaprootFreak merged 17 commits into
developfrom
01a09c2f-inbox-from-me

Conversation

@TaprootFreakAI

@TaprootFreakAI TaprootFreakAI commented Sep 13, 2026

Copy link
Copy Markdown
Collaborator

EN:
The Nachrichten list is incoming threads only. At the top you choose Direct, Contact, or Damus so private PMs and official tickets are not mixed. Sent bubbles stay filled on the right.

DE:
Die Nachrichten-Liste zeigt nur eingehende Threads. Oben wählt man Direkt, Kontakt oder Damus, damit private PNs und offizielle Tickets nicht gemischt sind. Gesendete Blasen bleiben rechts gefüllt.

Details

Paired API PR 21gifts/api#130. Default filter Direct. Linux goldens: screen-messages (Direct), state-messages-contact, state-messages-damus, sent-preview, empty. state-contact-success unchanged.

@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 13, 2026 20:55
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 05:08
@TaprootFreakAI TaprootFreakAI changed the title 01a09c2f - Show sent inbox previews as You, not as received 01a09c2f - Show incoming inbox threads, not outbound post replies Sep 14, 2026
@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 14, 2026 08:03
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 08:12
@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 14, 2026 08:49
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 09:07
@TaprootFreakAI TaprootFreakAI changed the title 01a09c2f - Show incoming inbox threads, not outbound post replies 01a09c2f - Show incoming inbox threads with clear sent vs received chrome Sep 14, 2026
@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 14, 2026 09:56
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 10:05
@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 14, 2026 10:36
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 11:07
@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 14, 2026 11:36
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 11:40
@TaprootFreakAI TaprootFreakAI changed the title 01a09c2f - Show incoming inbox threads with clear sent vs received chrome 01a09c2f - Filter the inbox by Direct, Contact, or Damus Sep 14, 2026
@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review September 14, 2026 12:39
@TaprootFreakAI
TaprootFreakAI marked this pull request as draft September 14, 2026 13:41
The Nachrichten list prefixes the viewer's last message with You:/Du:
instead of attributing it to the counterpart. Thread bubbles for fromMe
messages sit on the right and show You. Incoming last text is unchanged.
Tailwind class order and the sent-preview ternary must match the repo
Prettier config so lint stays green.
The list prefix is only visible on the conversation list. The post test
now clears ?c= and rerenders after All conversations so lastFromMe can
be observed.
The loader post test asserts the sent bubble (You, data-from-me) on the
thread. The You: list prefix is covered by InboxScreen tests.
Only fromMe messages get ml-8 self-end. Incoming cards stay the previous
full-width class so Linux visual baselines do not change.
The mock GET /conversations drops empty and outbound-only threads to
match the api. You: still prefixes lastFromMe on listed two-way rows.
member_platform rows with any message stay listed so ContactLoader can
open ?c= after send. Outbound-only Direct threads stay omitted.
POST /contact and ContactLoader look up member_platform, so a Direct
chat named 21.gifts is not reused and still listed after send.
Linux Chromium goldens now show You: on the list and a right-aligned
You bubble in the thread, in all four viewport/theme combos.
Incoming thread bubbles stay muted and left; sent bubbles and list
previews use filled app-btn on the right so direction is obvious.
Incoming thread cards stay full width so contact-success pixels do
not move. Only sent bubbles, sent-preview chips, and thread goldens
change. CONTRIBUTING forbids incidental baseline diffs.
The default Nachrichten baseline is the inbox overview: a private
Direct thread, the official Contact ticket, and Damus.
The Nachrichten list has a chooser at the top. Default Direct shows
private PMs only; Contact and Damus are separate.
Linux thread snapshots follow the unified design-system chrome on
develop.
The screen-variant catalog listed member profile variants twice.
Contact send again asserts the official thread URL and body.
Empty inbox copy is per origin and the Direct/Contact/Damus control
stays visible, same as the forum mode switcher.
The lastFromMe behavior spec omitted location on /me, so accountSchema
parse failed and hid the filter, then clicked Contact on the Direct
default. Use a Direct Bob row with lastFromMe, matching sent-preview.

Split the inbox empty-state table so Prettier accepts docs/ui.md.
@TaprootFreakAI

Copy link
Copy Markdown
Collaborator Author

EN:
Ready after 2 review passes.
Nachrichten has a top chooser: Direct, Contact, or Damus, so private PMs and official tickets are not mixed.

DE:
Bereit nach 2 Review-Durchläufen.
Nachrichten hat oben eine Wahl: Direkt, Kontakt oder Damus, damit private PNs und offizielle Tickets nicht gemischt sind.

Details

Head c45c4e42. Comment gate: no open review threads, no inline comments, no review summaries. mergeable: MERGEABLE. CI run 34868245664 success at this head (Check, E2E behavior, four visual combos).

Pass 1 (previous head): empty-state table overflowed Prettier; lastFromMe e2e omitted location on /me and clicked Contact on the Direct default. Pass 2: table split into Direct/Contact/Damus rows; lastFromMe e2e uses Direct Bob with location: null; rebased onto current develop; 0 findings.

Paired API PR is 21gifts/api#130 and should merge first. Human merge only.

@TaprootFreak
TaprootFreak merged commit 106e8ea into develop Sep 14, 2026
12 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.

2 participants