Skip to content

msglist: Adding 🙂 emoji reaction fails #1495

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
chrisbobbe opened this issue May 2, 2025 · 1 comment
Open

msglist: Adding 🙂 emoji reaction fails #1495

chrisbobbe opened this issue May 2, 2025 · 1 comment
Assignees
Labels
a-msglist The message-list screen, except what's label:a-content server-11 Things new in Zulip Server 11.0

Comments

@chrisbobbe
Copy link
Collaborator

chrisbobbe commented May 2, 2025

Reproduced on CZO just now:

  1. Long-press a message
  2. Tap the 🙂 emoji in the list of emoji at the top of the message action sheet
  3. See error dialog (unexpected):
Image
"zulip_version":"10.0-550-g2182f40bd5"
"zulip_feature_level":380
@chrisbobbe chrisbobbe changed the title msglist: Adding 🙂 emoji fails msglist: Adding 🙂 emoji reaction fails May 2, 2025
@chrisbobbe chrisbobbe added the a-msglist The message-list screen, except what's label:a-content label May 2, 2025
@gnprice
Copy link
Member

gnprice commented May 2, 2025

Yeah, we should fix this.

Presumably this is due to the recent server change to make U+1F642 🙂 be called :slight_smile: instead of :smile::

The fix will probably involve updating this code:

  static final _popularCandidates = _generatePopularCandidates();

  static List<EmojiCandidate> _generatePopularCandidates() {
    // …
    return [
      // This list should match web:
      //   https://github.com/zulip/zulip/blob/83a121c7e/web/shared/src/typeahead.ts#L22-L29
      candidate('1f44d', '👍', ['+1', 'thumbs_up', 'like']),
      candidate('1f389', '🎉', ['tada']),
      candidate('1f642', '🙂', ['smile']),

The first step will be some refactoring to make that no longer static — it'll need to depend on zulipFeatureLevel.

@gnprice gnprice added the server-11 Things new in Zulip Server 11.0 label May 2, 2025
@gnprice gnprice added this to the M5a: Launch blockers milestone May 2, 2025
@chrisbobbe chrisbobbe self-assigned this May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-msglist The message-list screen, except what's label:a-content server-11 Things new in Zulip Server 11.0
Projects
Status: No status
Development

No branches or pull requests

2 participants