Skip to content

fix emoji selection using the touchscreen in WhatsApp - #1179

Merged
sspanak merged 1 commit into
masterfrom
fix-whatsapp-emojis-3
Aug 1, 2026
Merged

fix emoji selection using the touchscreen in WhatsApp#1179
sspanak merged 1 commit into
masterfrom
fix-whatsapp-emojis-3

Conversation

@sspanak

@sspanak sspanak commented Aug 1, 2026

Copy link
Copy Markdown
Owner

No description provided.

@sspanak sspanak self-assigned this Aug 1, 2026
@sspanak sspanak added the bug Something isn't working label Aug 1, 2026
@sspanak
sspanak force-pushed the fix-whatsapp-emojis-3 branch from dbcc570 to d31c37c Compare August 1, 2026 13:38
@sspanak sspanak linked an issue Aug 1, 2026 that may be closed by this pull request
@sspanak
sspanak requested a review from Copilot August 1, 2026 13:38
@sspanak
sspanak marked this pull request as ready for review August 1, 2026 13:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Ready to approve

The change is small, localized, and consistent with existing WhatsApp-specific composing-text handling already present in AppHacks.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

This PR adjusts composing-text handling when committing emoji suggestions so WhatsApp reliably accepts the selected emoji when chosen via touchscreen.

Changes:

  • Refreshes (forces) the composing text to the current suggestion before finishing/committing in specific app + content conditions.
  • Adds a dedicated AppHacks predicate to centralize the “needs refresh before commit” decision (currently WhatsApp + graphic-only text).
File summaries
File Description
app/src/main/java/io/github/sspanak/tt9/ime/helpers/SuggestionOps.java Forces composing text refresh (via replaceComposingText) before commit/finish when AppHacks indicates it’s needed.
app/src/main/java/io/github/sspanak/tt9/hacks/AppHacks.java Adds shouldRefreshComposingTextBeforeCommit(...) to detect the WhatsApp + graphic-text case that needs special handling.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@sspanak
sspanak merged commit 2e4bc0e into master Aug 1, 2026
7 checks passed
@sspanak
sspanak deleted the fix-whatsapp-emojis-3 branch August 1, 2026 13:46
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.

Two recent problems

2 participants