Page MenuHomePhabricator
Differential D14309

[native] Cancel animations when gesture is started in FullScreenViewModal
ClosedPublic
Actions

Authored by angelika on Feb 6 2025, 5:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 5, 8:24 PM
Unknown Object (File)
Sat, Apr 5, 3:08 PM
Unknown Object (File)
Sat, Apr 5, 12:08 PM
Unknown Object (File)
Sat, Apr 5, 11:10 AM
Unknown Object (File)
Fri, Apr 4, 8:06 PM
Unknown Object (File)
Fri, Apr 4, 7:59 PM
Unknown Object (File)
Fri, Apr 4, 5:09 PM
Unknown Object (File)
Fri, Apr 4, 1:40 PM
View All 62 Files
Subscribers
ashoat

Details

Reviewers
tomek
Commits
rCOMM3e75cc6b16f4: [native] Cancel animations when gesture is started in FullScreenViewModal
Summary

https://linear.app/comm/issue/ENG-8150/convert-fullscreenviewmodal-to-reanimated-2-syntax

As mentioned here: https://phab.comm.dev/D14305 in the original implementation there are a couple of places that stop animation if gesture is active. In this diff I added this logic.

The builds will fail due to some leftover reanimated v1 code. I'll remove it in the next diffs for diffs readability.

Depends on D14308

Test Plan

Test after migration: try to start a gesture when animation is in progress

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

angelika created this revision.Feb 6 2025, 5:42 PM
angelika held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptFeb 6 2025, 5:42 PM
angelika added a child revision: D14310: [native] Remove leftover Reanimated v1 code in FullScreenViewModal.Feb 6 2025, 5:42 PM
Harbormaster failed remote builds in B33424: Diff 46953!Feb 6 2025, 5:54 PM
angelika edited the summary of this revision. (Show Details)Feb 10 2025, 1:02 AM
angelika updated this revision to Diff 47087.Feb 12 2025, 8:53 PM

Rebase & bugfixes

angelika published this revision for review.Feb 12 2025, 9:10 PM
angelika edited the summary of this revision. (Show Details)
Harbormaster failed remote builds in B33503: Diff 47087!Feb 12 2025, 9:21 PM
tomek accepted this revision.Feb 17 2025, 4:00 AM
tomek mentioned this in D14308: [native] Migrate fling functionality in FullScreenViewModal.

Worth checking if this diff causes an issue from a comment https://phab.comm.dev/D14308#399134.

This revision is now accepted and ready to land.Feb 17 2025, 4:01 AM
ashoat mentioned this in D14305: [native] Migrate double tap gesture handling in FullScreenViewModal.Feb 19 2025, 4:34 PM
ashoat added a comment.Feb 19 2025, 4:42 PM

I'd be surprised if this diff was the cause, as here we cancel animations at the start of a pan or pinch, but in the video we see an issue at the conclusion of a pan.

angelika updated this revision to Diff 47191.Feb 20 2025, 2:15 PM

Rebase & bugfixes

Harbormaster failed remote builds in B33572: Diff 47191!Feb 20 2025, 2:42 PM
angelika updated this revision to Diff 47203.Feb 21 2025, 8:08 PM

Rebase

Harbormaster failed remote builds in B33583: Diff 47203!Feb 21 2025, 8:23 PM
angelika updated this revision to Diff 47209.Feb 21 2025, 10:14 PM

Fix rebase

Harbormaster failed remote builds in B33589: Diff 47209!Feb 21 2025, 10:28 PM
angelika updated this revision to Diff 47225.Feb 24 2025, 3:53 AM

Rebase

Harbormaster failed remote builds in B33605: Diff 47225!Feb 24 2025, 4:19 AM
This revision was landed with ongoing or failed builds.Feb 25 2025, 2:48 AM
Closed by commit rCOMM3e75cc6b16f4: [native] Cancel animations when gesture is started in FullScreenViewModal (authored by angelika). · Explain Why
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMM3e75cc6b16f4: [native] Cancel animations when gesture is started in FullScreenViewModal.

Revision Contents
Changeset List

PathSize
native/
components/
20 lines

Diff 47253

View Options

native/components/full-screen-view-modal.react.js

// @flow // @flow
import t from 'tcomb'; import t from 'tcomb';
import type { TInterface, TUnion } from 'tcomb'; import type { TInterface, TUnion } from 'tcomb';
import { mixedRawThreadInfoValidator } from 'lib/permissions/minimally-encoded-thread-permissions-validators.js'; import { mixedRawThreadInfoValidator } from 'lib/permissions/minimally-encoded-raw-thread-info-validators.js';
import { mediaValidator } from 'lib/types/media-types.js'; import { mediaValidator } from 'lib/types/media-types.js';
import { import {
rawMessageInfoValidator, rawMessageInfoValidator,
messageTruncationStatusesValidator, messageTruncationStatusesValidator,
} from 'lib/types/message-types.js'; } from 'lib/types/message-types.js';
import { userSurfacedPermissionValidator } from 'lib/types/thread-permission-types.js'; import { userSurfacedPermissionValidator } from 'lib/types/thread-permission-types.js';
import { threadTypes } from 'lib/types/thread-types-enum.js'; import { threadTypes } from 'lib/types/thread-types-enum.js';
import { import {
▲ Show 20 Lines • Show All 325 Lines • Show Last 20 Lines