emoji scroll and zoom fix

[ragnep]

Summary by CodeRabbit

• Bug Fixes

  • Fixed touch event handling in the mobile emoji picker to prevent unintended interactions with underlying content.

• Style

  • Adjusted emoji picker font sizing for better readability on mobile devices.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

Three files modified to enhance mobile emoji picker behavior: touch move event propagation is prevented in two picker wrapper components, and emoji picker font size styling is added to the global stylesheet.

Changes

Cohort / File(s)	Summary
Mobile Emoji Picker Touch Handling components/waves/CreateDropEmojiPicker.tsx, components/waves/drops/WaveDropActionsAddReaction.tsx	Added onTouchMove={(e) => e.stopPropagation()} handler to mobile emoji picker wrapper divs to prevent touch move events from bubbling to parent elements.
Emoji Picker Styling styles/globals.scss	Added CSS rule for em-emoji-picker element setting custom property --font-size to 17px.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• Touchscreen v2 #1851: Refactors touch-move gating and long-press logic in mobile emoji picker interactions, complementing the touch event propagation fixes.
• Reactions dialog #1718: Modifies WaveDropActionsAddReaction to handle touch event propagation in the mobile emoji picker, directly overlapping with this PR's changes.

Suggested reviewers

• simo6529
• GelatoGenesis

Poem

🐰✨ A gentle paw upon the screen,
Touch events now stay pristine,
No bubbles float where they shouldn't be,
Emoji picker flows wild and free!
With font sizes set just right,
Mobile interactions take flight! 🎉

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'emoji scroll and zoom fix' is vague and does not clearly describe the specific changes made. The actual changes involve adding touch event propagation handling and CSS font-size property, which are not directly related to 'scroll and zoom'.	Provide a more specific title that accurately describes the changes, such as 'Stop touch propagation in emoji picker mobile wrapper' or 'Add touch event handling to mobile emoji picker'.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch app-wave-ux-fixes-v2

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud