Skip to content

fix(web): localize collaboration markers and reuse loading state - #325

Merged
cursor[bot] merged 2 commits into
elie222:mainfrom
luinbytes:fix/collaboration-marker-review
Aug 27, 2026
Merged

fix(web): localize collaboration markers and reuse loading state#325
cursor[bot] merged 2 commits into
elie222:mainfrom
luinbytes:fix/collaboration-marker-review

Conversation

@luinbytes

Copy link
Copy Markdown
Contributor

Summary

  • display compact centered collaboration markers with the peer avatar and the complete localized label
  • render active bot glyphs through the existing LoadingState primitive
  • preserve the avatar-style context merged into current main
  • regenerate locale catalogs from source

This is the verified successor to #309/#320. The maintainer-owned #309 branch copied the conflict merge but not the later Greptile fixes.

Verification

  • pnpm exec biome check apps/web/src/pages/Shell.tsx apps/web/src/components/beautiful-ui/CollaborationMarker.tsx apps/web/src/components/beautiful-ui/CollaborationMarker.test.tsx apps/web/src/components/beautiful-ui/primitives.tsx apps/web/src/locales
  • pnpm --filter @rakazo/web check
  • pnpm --filter @rakazo/web test (20 files, 120 tests)

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

@luinbytes is attempting to deploy a commit to the Inbox Zero Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 34 minutes.

View limit details

Limit details: You’ve used all 4 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f85fa399-fe32-49a3-86a7-d9073c386e95

📥 Commits

Reviewing files that changed from the base of the PR and between 43e96cb and 0d5b388.

📒 Files selected for processing (10)
  • apps/web/src/components/beautiful-ui/CollaborationMarker.test.tsx
  • apps/web/src/components/beautiful-ui/CollaborationMarker.tsx
  • apps/web/src/components/beautiful-ui/primitives.tsx
  • apps/web/src/locales/de/messages.po
  • apps/web/src/locales/en/messages.po
  • apps/web/src/locales/hi/messages.po
  • apps/web/src/locales/ko/messages.po
  • apps/web/src/locales/pt-BR/messages.po
  • apps/web/src/locales/tr/messages.po
  • apps/web/src/pages/Shell.tsx

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.

@greptile-apps

greptile-apps Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR localizes collaboration markers, preserves peer avatar identity, and reuses the shared loading-state primitive for active bot indicators.

  • Extracts collaboration markers and active bot glyphs into a dedicated component.
  • Uses canonical bot IDs and resolved peer colors for avatar rendering.
  • Prevents the reconnect timer from continuously updating a live region.
  • Adds localized active-bot labels and focused rendering tests.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/web/src/components/beautiful-ui/primitives.tsx Separates timer and custom-indicator loading states so only the static indicator label is exposed as a status.
apps/web/src/components/beautiful-ui/CollaborationMarker.tsx Adds compact collaboration markers with identity-aware avatars and a shared active-bot loading indicator.
apps/web/src/pages/Shell.tsx Resolves transcript bot metadata, supplies canonical avatar identities, and constructs localized working-state labels.
apps/web/src/components/beautiful-ui/CollaborationMarker.test.tsx Covers accessible marker labeling, avatar rendering, and active-bot status semantics.
apps/web/src/locales/en/messages.po Adds the extracted active-bot status messages, mirrored across the updated locale catalogs.

Reviews (2): Last reviewed commit: "fix(web): preserve avatar identity and l..." | Re-trigger Greptile

Comment thread apps/web/src/components/beautiful-ui/primitives.tsx Outdated
Comment thread apps/web/src/components/beautiful-ui/CollaborationMarker.tsx Outdated
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Playwright screenshots

Open screenshot gallery · Dashboard · CI run

Updated for commit 0d5b388.

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
rakazo-www Skipped Skipped Aug 27, 2026 2:53pm

Request Review

Display compact centered collaboration markers with the peer avatar and
full localized label, and render active bot glyphs through LoadingState
with an optional indicator. Rebased onto current main while preserving
transcript follow behavior.

Co-authored-by: Elie Steinbock <elie222@users.noreply.github.com>
@cursor
cursor Bot force-pushed the fix/collaboration-marker-review branch from 6030b12 to 270efaa Compare August 27, 2026 14:51
@cursor
cursor Bot merged commit 26a1184 into elie222:main Aug 27, 2026
14 checks passed
@greptile-apps

greptile-apps Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Want your agent to iterate on Greptile's feedback? Try greploops.

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