Skip to content

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Nov 13, 2025

Purpose

Fix duplicated emoji when pasting an emoji at the beginning of a sub‑doc title. The pasted emoji must stay in the title and must not be turned into the sub‑doc icon.

issue : 1592

copypasteemojis.mp4

Proposal

  • Preserve pasted leading emoji in title; prevent icon capture via zero‑width space in DocTitle.tsx
  • Keep existing icon unchanged; icon only changes via ā€œAdd/Remove emojiā€
  • Make emoji detection ignore leading spaces in getEmojiAndTitle

@Ovgodd Ovgodd requested a review from AntoLC November 13, 2025 13:38
@Ovgodd Ovgodd self-assigned this Nov 13, 2025
@Ovgodd Ovgodd added the bug Something isn't working label Nov 13, 2025
@Ovgodd Ovgodd changed the title ✨(frontend) prevent duplicate emoji when used as first character in t… šŸ›(frontend) prevent duplicate emoji when used as first character in t… Nov 13, 2025
@Ovgodd Ovgodd force-pushed the fix/1592-copy-paste-emoji-in-title branch 2 times, most recently from 5e4f415 to 7ed8ea5 Compare November 13, 2025 13:41
@github-actions
Copy link

github-actions bot commented Nov 13, 2025

Size Change: +63 B (0%)

Total Size: 4.07 MB

Filename Size Change
apps/impress/out/_next/static/b2df1b37/_buildManifest.js 0 B -884 B (removed) šŸ†
apps/impress/out/_next/static/792e84ec/_buildManifest.js 884 B +884 B (new file) šŸ†•

compressed-size-action

@Ovgodd Ovgodd marked this pull request as ready for review November 13, 2025 13:45
@Ovgodd Ovgodd force-pushed the fix/1592-copy-paste-emoji-in-title branch from 7ed8ea5 to 0f4a14d Compare November 13, 2025 13:49
…itle

ensures icon and title are visually distinct in sub-document headers

Signed-off-by: Cyril <c.gromoff@gmail.com>
@Ovgodd Ovgodd force-pushed the fix/1592-copy-paste-emoji-in-title branch from 0f4a14d to 135ea9a Compare November 13, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants