Skip to content

refactor(meet): use plain textarea and gemoji instead of pulling editor#195

Merged
BreadGenie merged 3 commits into
frappe:developfrom
BreadGenie:refactor/meet-chat-textarea-emoji
Jul 11, 2026
Merged

refactor(meet): use plain textarea and gemoji instead of pulling editor#195
BreadGenie merged 3 commits into
frappe:developfrom
BreadGenie:refactor/meet-chat-textarea-emoji

Conversation

@BreadGenie

@BreadGenie BreadGenie commented Jul 11, 2026

Copy link
Copy Markdown
Member

also remove the remnant package.union.json file from the suite migration

…suggest

Drop frappe-ui/editor from the meet chat composer so the meet chunk no longer
pulls CommentKit/ProseMirror for plain-text input. Use a textarea with colon
emoji autocomplete (gemoji) and align the people panel close button with chat.
Remember selected emojis and surface them first on empty `:` queries,
padding with the default top matches so the menu always shows five items.
Drop the suite-unification dependency snapshot; package.json is the
source of truth and nothing referenced the union file.
@greptile-apps

greptile-apps Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

Safe to merge — the refactor is self-contained, well-tested, and removes a heavyweight dependency without changing the wire protocol.

The emoji utility is fully unit-tested, edge cases (URL colon, empty query, recents dedup) are handled, and the textarea swap is straightforward. No data-loss or auth paths are touched.

No files require special attention.

Reviews (1): Last reviewed commit: "chore(frontend): remove obsolete package..." | Re-trigger Greptile

@BreadGenie BreadGenie merged commit 99ef6a2 into frappe:develop Jul 11, 2026
8 checks passed
@BreadGenie BreadGenie deleted the refactor/meet-chat-textarea-emoji branch July 11, 2026 13:39
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.

1 participant