Skip to content

Commit 96aa1f2

Browse files
committed
Revert "fix(ios): prevent error while overriding N gestures"
This reverts commit bb04710.
1 parent 8698849 commit 96aa1f2

File tree

7 files changed

+10
-24
lines changed

7 files changed

+10
-24
lines changed

packages/gesturehandler/platforms/ios/src/GestureHandlerManager.m

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,10 +88,6 @@ - (void)dropGestureHandler:(NSNumber *)handlerTag
8888
[_registry dropHandlerWithTag:handlerTag];
8989
}
9090

91-
- (void)detachGestureHandler:(NSNumber *)handlerTag
92-
{
93-
[_registry detachHandlerWithTag:handlerTag];
94-
}
9591
//- (void)handleSetJSResponder:(NSNumber *)viewTag blockNativeResponder:(NSNumber *)blockNativeResponder
9692
//{
9793
// if ([blockNativeResponder boolValue]) {

packages/gesturehandler/platforms/ios/src/GestureHandlerRegistry.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,5 @@
1414
- (void)registerGestureHandler:(nonnull GestureHandler *)gestureHandler;
1515
- (void)attachHandlerWithTag:(nonnull NSNumber *)handlerTag toView:(nonnull UIView *)view;
1616
- (void)dropHandlerWithTag:(nonnull NSNumber *)handlerTag;
17-
- (void)detachHandlerWithTag:(nonnull NSNumber *)handlerTag;
18-
- (void)detachHandlerWithTag:(nonnull NSNumber *)handlerTag;
1917

2018
@end

packages/gesturehandler/platforms/ios/src/GestureHandlerRegistry.m

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,4 @@ - (void)dropHandlerWithTag:(NSNumber *)handlerTag
4646
[_handlers removeObjectForKey:handlerTag];
4747
}
4848

49-
- (void)detachHandlerWithTag:(NSNumber *)handlerTag
50-
{
51-
GestureHandler *handler = _handlers[handlerTag];
52-
[handler unbindFromView];
53-
}
5449
@end

src/gesturehandler/gesturehandler.d.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ export enum FlingDirection {
1010
DIRECTION_LEFT,
1111
DIRECTION_UP,
1212
DIRECTION_DOWN,
13-
DIRECTION_RIGHT
13+
DIRECTION_RIGHT,
1414
}
1515

1616
export abstract class BaseNative<T, U extends {}> extends Observable {
@@ -65,7 +65,7 @@ export abstract class Handler<T, U extends HandlerOptions> extends BaseNative<T,
6565
getView(): View;
6666
cancel();
6767
attachToView(view: View);
68-
detachFromView(view?: View, drop?: boolean);
68+
detachFromView(view?: View);
6969
}
7070
export interface TapGestureHandlerOptions extends HandlerOptions {
7171
numberOfTaps?: number;

src/gesturehandler/gesturehandler.ios.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -195,14 +195,14 @@ export class Handler<T extends GestureHandler, U extends HandlerOptions> extends
195195
return;
196196
}
197197
if (this.attachedView) {
198-
this.detachFromView(this.attachedView, false);
198+
this.detachFromView(this.attachedView);
199199
}
200200
this.attachedView = view;
201201
this.delegate = HandlerDelegate.initWithOwner(new WeakRef(this));
202202
this.native.delegate = this.delegate;
203203
this.manager.get().attachGestureHandler(this, view);
204204
}
205-
detachFromView(view?: View, drop = true) {
205+
detachFromView(view?: View) {
206206
if ((view && view !== this.attachedView) || !this.attachedView) {
207207
return;
208208
}
@@ -212,7 +212,7 @@ export class Handler<T extends GestureHandler, U extends HandlerOptions> extends
212212
}
213213
const tag = this.native.tag;
214214
this.delegate = this.native.delegate = null;
215-
this.manager.get().detachGestureHandler(tag, this.attachedView, drop);
215+
this.manager.get().detachGestureHandler(tag, this.attachedView);
216216
this.attachedView = null;
217217
}
218218
getTag() {
@@ -274,11 +274,9 @@ export class Manager extends ManagerBase {
274274
dispose: onDispose
275275
});
276276
}
277-
detachGestureHandler(handlerTag: number, view: View, drop = true) {
278-
if (drop) {
277+
detachGestureHandler(handlerTag: number, view: View) {
278+
if (view.nativeView) {
279279
this.manager.dropGestureHandler(handlerTag);
280-
} else {
281-
this.manager.detachGestureHandler(handlerTag);
282280
}
283281
if (view) {
284282
const viewListeners = this.viewListeners.get(view);

src/gesturehandler/gestures_override.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ export class GesturesObserver {
6060
}
6161

6262
public disconnect() {
63-
this._detach(true);
63+
this._detach();
6464

6565
if (this.target) {
6666
this.target.off('loaded', this._onTargetLoaded);
@@ -123,12 +123,12 @@ export class GesturesObserver {
123123
}
124124
}
125125

126-
private _detach(drop = false) {
126+
private _detach() {
127127
if (this.gestureHandler) {
128128
// dont detach events. It will be done on dispose
129129
// this.gestureHandler.off(GestureHandlerStateEvent);
130130
// this.gestureHandler.off(GestureHandlerTouchEvent);
131-
this.gestureHandler.detachFromView(this.target, drop);
131+
this.gestureHandler.detachFromView(this.target);
132132
}
133133
this._notifyTouch = false;
134134
this._eventData = {};

src/gesturehandler/typings/ios.d.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,6 @@ declare class GestureHandlerManager extends NSObject {
157157
createGestureHandlerTagConfig(handlerName: string, handlerTag: number, config: NSDictionary<any, any>): GestureHandler;
158158

159159
dropGestureHandler(handlerTag: number): void;
160-
detachGestureHandler(handlerTag: number): void;
161160

162161
updateGestureHandlerConfig(handlerTag: number, config: NSDictionary<any, any>): void;
163162
}

0 commit comments

Comments
 (0)