Skip to content

fix(mobile): highlight full multi-word mentions#1762

Merged
brow merged 3 commits into
mainfrom
tomb/mobile-full-name-mentions
Jul 12, 2026
Merged

fix(mobile): highlight full multi-word mentions#1762
brow merged 3 commits into
mainfrom
tomb/mobile-full-name-mentions

Conversation

@brow

@brow brow commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

What changed?

  • Match the non-breaking-space form used by mobile markdown preprocessing when resolving multi-word mention aliases.
  • Add regression coverage proving a full display name renders as one highlighted mention instead of duplicating the surname.

Why?

Mobile currently highlights only the first name in a full-name mention and leaves a duplicate surname as plain text.

How is it tested?

Added tests:

Focused mobile validation and static analysis pass.

Co-authored-by: npub1shglkdhngx3hrnhf4gf8vhpqdrmeludctechdvpwd3988zzs7ncq2cmtxu <85d1fb36f341a371cee9aa12765c2068f79ff1b85e7176b02e6c4a738850f4f0@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1shglkdhngx3hrnhf4gf8vhpqdrmeludctechdvpwd3988zzs7ncq2cmtxu <85d1fb36f341a371cee9aa12765c2068f79ff1b85e7176b02e6c4a738850f4f0@sprout-oss.stage.blox.sqprod.co>
@brow

brow commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@brow

brow commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

@builderbot review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@brow brow marked this pull request as ready for review July 12, 2026 03:15
@brow brow requested a review from a team as a code owner July 12, 2026 03:15
@brow brow enabled auto-merge (squash) July 12, 2026 03:15
Comment thread mobile/lib/features/channels/message_content.dart Outdated
Co-authored-by: npub1shglkdhngx3hrnhf4gf8vhpqdrmeludctechdvpwd3988zzs7ncq2cmtxu <85d1fb36f341a371cee9aa12765c2068f79ff1b85e7176b02e6c4a738850f4f0@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1shglkdhngx3hrnhf4gf8vhpqdrmeludctechdvpwd3988zzs7ncq2cmtxu <85d1fb36f341a371cee9aa12765c2068f79ff1b85e7176b02e6c4a738850f4f0@sprout-oss.stage.blox.sqprod.co>
Comment thread mobile/lib/features/channels/message_content.dart Outdated
Co-authored-by: npub1shglkdhngx3hrnhf4gf8vhpqdrmeludctechdvpwd3988zzs7ncq2cmtxu <85d1fb36f341a371cee9aa12765c2068f79ff1b85e7176b02e6c4a738850f4f0@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1shglkdhngx3hrnhf4gf8vhpqdrmeludctechdvpwd3988zzs7ncq2cmtxu <85d1fb36f341a371cee9aa12765c2068f79ff1b85e7176b02e6c4a738850f4f0@sprout-oss.stage.blox.sqprod.co>
@wesbillman

Copy link
Copy Markdown
Collaborator

@codex please review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

Reviewed commit: b0faeadf32

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@brow brow merged commit 92bb959 into main Jul 12, 2026
25 checks passed
@brow brow deleted the tomb/mobile-full-name-mentions branch July 12, 2026 15:14
wpfleger96 pushed a commit that referenced this pull request Jul 12, 2026
* origin/main:
  fix(desktop): correct provider and model handling in agent config dialogs (#1764)
  fix(desktop): cascade persona deletes and restart agents on global config save (#1766)
  fix(desktop): make doctor installs retryable with per-runtime progress and auth status (#1765)
  fix(codex): swap to @agentclientprotocol/codex-acp 1.x + detect outdated adapter (#1750)
  Fix oversized mobile Manage channel sheet (#1774)
  fix(desktop): show thread replies loader (#1773)
  fix(build): allow vendored opus to configure under CMake 4 (#1763)
  fix(desktop): restore multi-image mosaic galleries (#1769)
  Group channel membership events (#1713)
  Fix desktop launch motion and reaction spacing (#1717)
  fix(desktop): align sidebar search across themes (#1712)
  fix(desktop): Buzz theme flicker, white bar & accent-picker motion (#1681)
  fix(mobile): highlight full multi-word mentions (#1762)
  [BOT-1264] fix(mobile): #channel tags not tappable (#1695)
  feat(desktop): surface Team Instructions as distinct observer section (#1759)

# Conflicts:
#	desktop/playwright.config.ts
wpfleger96 pushed a commit that referenced this pull request Jul 12, 2026
* origin/main:
  fix(desktop): correct provider and model handling in agent config dialogs (#1764)
  fix(desktop): cascade persona deletes and restart agents on global config save (#1766)
  fix(desktop): make doctor installs retryable with per-runtime progress and auth status (#1765)
  fix(codex): swap to @agentclientprotocol/codex-acp 1.x + detect outdated adapter (#1750)
  Fix oversized mobile Manage channel sheet (#1774)
  fix(desktop): show thread replies loader (#1773)
  fix(build): allow vendored opus to configure under CMake 4 (#1763)
  fix(desktop): restore multi-image mosaic galleries (#1769)
  Group channel membership events (#1713)
  Fix desktop launch motion and reaction spacing (#1717)
  fix(desktop): align sidebar search across themes (#1712)
  fix(desktop): Buzz theme flicker, white bar & accent-picker motion (#1681)
  fix(mobile): highlight full multi-word mentions (#1762)
  [BOT-1264] fix(mobile): #channel tags not tappable (#1695)
  feat(desktop): surface Team Instructions as distinct observer section (#1759)

Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>

# Conflicts:
#	desktop/playwright.config.ts
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