Content-Length: 3694 | pFad | http://github.com/NativeScript/NativeScript/pull/10690.diff
thub.com diff --git a/packages/core/ui/layouts/layout-base-common.ts b/packages/core/ui/layouts/layout-base-common.ts index 5e760d2d9e..0a240601ed 100644 --- a/packages/core/ui/layouts/layout-base-common.ts +++ b/packages/core/ui/layouts/layout-base-common.ts @@ -49,14 +49,14 @@ export class LayoutBaseCommon extends CustomLayoutView implements LayoutBaseDefi this._registerLayoutChild(child); } - public insertChild(child: View, atIndex: number): void { - if (atIndex < 0) { - throw new Error('Cannot insert a child to a negative index.'); + public insertChild(child: View, atIndex: number): boolean { + if (atIndex > -1) { + this._subViews.splice(atIndex, 0, child); + this._addView(child, atIndex); + this._registerLayoutChild(child); + return true; } - - this._subViews.splice(atIndex, 0, child); - this._addView(child, atIndex); - this._registerLayoutChild(child); + return false; } public removeChild(child: View): void { diff --git a/packages/core/ui/layouts/layout-base.d.ts b/packages/core/ui/layouts/layout-base.d.ts index f94eb7b835..a9ffd9e078 100644 --- a/packages/core/ui/layouts/layout-base.d.ts +++ b/packages/core/ui/layouts/layout-base.d.ts @@ -34,7 +34,7 @@ export class LayoutBase extends CustomLayoutView { * @param view The view to be added to the end of the children array. * @param atIndex The insertion index. */ - insertChild(child: View, atIndex: number): void; + insertChild(child: View, atIndex: number): boolean; /** * Removes the specified view from the children array. diff --git a/packages/core/ui/layouts/layout-base.ios.ts b/packages/core/ui/layouts/layout-base.ios.ts index 13f8fa3200..7c70039639 100644 --- a/packages/core/ui/layouts/layout-base.ios.ts +++ b/packages/core/ui/layouts/layout-base.ios.ts @@ -11,9 +11,12 @@ export class LayoutBase extends LayoutBaseCommon { this.requestLayout(); } - public insertChild(child: View, atIndex: number): void { - super.insertChild(child, atIndex); - this.requestLayout(); + public insertChild(child: View, atIndex: number): boolean { + if (super.insertChild(child, atIndex)) { + this.requestLayout(); + return true; + } + return false; } public removeChild(child: View): void { diff --git a/packages/core/ui/layouts/root-layout/index.android.ts b/packages/core/ui/layouts/root-layout/index.android.ts index d85e292c25..6ad9bcfe98 100644 --- a/packages/core/ui/layouts/root-layout/index.android.ts +++ b/packages/core/ui/layouts/root-layout/index.android.ts @@ -8,20 +8,23 @@ import { LinearGradient } from '../../styling/linear-gradient'; export * from './root-layout-common'; export class RootLayout extends RootLayoutBase { - insertChild(view: View, atIndex: number): void { - super.insertChild(view, atIndex); - if (!view.hasGestureObservers()) { - // block tap events from going through to layers behind the view - if (view.nativeViewProtected) { - view.nativeViewProtected.setOnTouchListener( - new android.view.View.OnTouchListener({ - onTouch: function (view, event) { - return true; - }, - }), - ); + insertChild(view: View, atIndex: number): boolean { + if (super.insertChild(view, atIndex)) { + if (!view.hasGestureObservers()) { + // block tap events from going through to layers behind the view + if (view.nativeViewProtected) { + view.nativeViewProtected.setOnTouchListener( + new android.view.View.OnTouchListener({ + onTouch: function (view, event) { + return true; + }, + }), + ); + } } + return true; } + return false; } removeChild(view: View): void { if (view.hasGestureObservers() && view.nativeViewProtected) {Fetched URL: http://github.com/NativeScript/NativeScript/pull/10690.diff
Alternative Proxies: