remove animation in gallery for touchscreens

[simo6529]

Summary by CodeRabbit

Release Notes

• Performance

  • Optimized loading animations on touch-screen devices to reduce unnecessary visual effects and improve responsiveness.

• UI Improvements

  • Enhanced media gallery and modal component layouts for better touch device compatibility.
  • Minor structural refinements for consistent visual presentation.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Three components are updated to conditionally disable loading pulse animations on touch-screen devices by integrating a device detection hook. Additionally, an unnecessary type cast for voting credit type is removed from the leaderboard component.

Changes

Cohort / File(s)	Summary
Touch-device animation detection components/drops/view/item/content/media/DropListItemContentMediaImage.tsx, components/drops/view/item/content/media/MediaDisplayImage.tsx	Added useDeviceInfo hook to detect touch screens and conditionally apply pulse animation class to loading placeholders; animation is disabled on touch devices. Minor JSX reordering and className adjustments in modal structure.
Type casting removal and transition logic components/waves/leaderboard/gallery/WaveLeaderboardGalleryItem.tsx	Removed explicit ApiWaveCreditType casting; voting_credit_type now used directly without cast. Inverted transition class calculation logic to align with hasTouchScreen checks. Removed unused ApiWaveCreditType import.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Seize sentry error fixes #1668 — Modifies the useDeviceInfo hook implementation while this PR consumes it across multiple components.
• Added title leaderboard grid card and improved ui #1635 — Updates the same WaveLeaderboardGalleryItem component with similar touch-device-driven animation logic changes.
• Reactions dialog #1718 — Adjusts conditional UI behavior based on touch-device detection across multiple components.

Suggested reviewers

• ragnep
• prxt6529
• analyticsflowee

Poem

🐰 A touch upon the screen, no pulse to see—
The rabbit knows: keep animations free!
One cast removed, the logic turns around,
Device-aware animations, smooth and sound. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'remove animation in gallery for touchscreens' directly summarizes the main change: conditionally disabling pulse animations on touch devices across gallery-related components.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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