Fix ios placeholder implementation (#4067)

* Fix https://github.com/NativeScript/NativeScript/issues/4043

* Merge ios & android implementation
This commit is contained in:
Hristo Hristov
2017-04-26 14:27:24 +03:00
committed by GitHub
parent 826c922a98
commit 502a37d27c
4 changed files with 47 additions and 52 deletions

View File

@@ -1,7 +1,6 @@
import * as TKUnit from "../../TKUnit";
// >> article-creating-view
import * as platform from "tns-core-modules/platform";
//var utils = require("utils/utils");
import { isIOS, isAndroid } from "tns-core-modules/platform";
import * as utils from "tns-core-modules/utils/utils";
import * as helper from "../helper";
import * as viewModule from "tns-core-modules/ui/core/view";
@@ -11,31 +10,29 @@ import * as placeholderModule from "tns-core-modules/ui/placeholder";
// << article-require-placeholder-module
function creatingView(args) {
var nativeView;
if (platform.device.os === platform.platformNames.ios) {
nativeView = UITextView.new();
nativeView.text = "Native";
} else if (platform.device.os === platform.platformNames.android) {
nativeView = new android.widget.TextView(utils.ad.getApplicationContext());
nativeView.setText("Native");
}
let nativeView;
if (isIOS) {
nativeView = UITextView.new();
nativeView.text = "Native";
} else if (isAndroid) {
nativeView = new android.widget.TextView(utils.ad.getApplicationContext());
nativeView.setText("Native");
}
args.view = nativeView;
args.view = nativeView;
}
exports.creatingView = creatingView;
// << article-creating-view
export function test_placeholder_creatingView() {
var nativeView;
var p = new placeholderModule.Placeholder();
p.id = "test";
const p = new placeholderModule.Placeholder();
p.on(placeholderModule.Placeholder.creatingViewEvent, (args: placeholderModule.CreateViewEventData) => {
if (platform.device.os === platform.platformNames.ios) {
nativeView = UITextView.new();
let nativeView;
if (isIOS) {
nativeView = UITextView.new();
nativeView.text = "Native";
} else if (platform.device.os === platform.platformNames.android) {
} else if (isAndroid) {
nativeView = new android.widget.TextView(utils.ad.getApplicationContext());
nativeView.setText("Native");
}
@@ -43,21 +40,24 @@ export function test_placeholder_creatingView() {
args.view = nativeView;
});
if (platform.device.os === platform.platformNames.ios) {
TKUnit.assert(p.ios instanceof UITextView, "ios property should be UITextView. Current value: " + p.ios);
} else if (platform.device.os === platform.platformNames.android) {
p._emit("creatingView");
TKUnit.assert(nativeView instanceof android.widget.TextView, "Native view should be android.widget.TextView. Current value: " + nativeView);
}
function testAction(views: Array<viewModule.View>) {
if (isIOS) {
TKUnit.assert(p.nativeView instanceof UITextView, "nativeView property should be UITextView. Current value: " + p.nativeView);
} else if (isAndroid) {
TKUnit.assert(p.nativeView instanceof android.widget.TextView, "Native view should be android.widget.TextView. Current value: " + p.nativeView);
}
};
helper.buildUIAndRunTest(p, testAction);
}
export function test_placeholder_will_not_crash_wihout_creatingView() {
var p = new placeholderModule.Placeholder();
const p = new placeholderModule.Placeholder();
function testAction(views: Array<viewModule.View>) {
if (platform.device.os === platform.platformNames.ios) {
if (isIOS) {
TKUnit.assert(p.ios === undefined, "ios property should be undefined. Current value: " + p.ios);
} else if (platform.device.os === platform.platformNames.android) {
} else if (isAndroid) {
TKUnit.assert(p.android === undefined, "android view should be undefined. Current value: " + p.android);
}
};

View File

@@ -5,7 +5,7 @@ export class Placeholder extends View implements PlaceholderDefinition {
public static creatingViewEvent = "creatingView";
public createNativeView() {
let args = <CreateViewEventData>{ eventName: Placeholder.creatingViewEvent, object: this, view: undefined, context: this._context };
const args = <CreateViewEventData>{ eventName: Placeholder.creatingViewEvent, object: this, view: undefined, context: this._context };
this.notify(args);
return <android.view.View>args.view;
}

View File

@@ -1,17 +0,0 @@
import { Placeholder as PlaceholderDefinition, CreateViewEventData } from "."
import { View } from "../core/view"
export class Placeholder extends View implements PlaceholderDefinition {
public static creatingViewEvent = "creatingView";
private _ios: UIView;
get ios(): UIView {
if (!this._ios) {
var args = <CreateViewEventData>{ eventName: Placeholder.creatingViewEvent, object: this, view: undefined, context: undefined };
super.notify(args);
this.nativeView = this._ios = args.view;
}
return this._ios;
}
}

View File

@@ -0,0 +1,12 @@
import { Placeholder as PlaceholderDefinition, CreateViewEventData } from "."
import { View } from "../core/view"
export class Placeholder extends View implements PlaceholderDefinition {
public static creatingViewEvent = "creatingView";
public createNativeView() {
const args = <CreateViewEventData>{ eventName: Placeholder.creatingViewEvent, object: this, view: undefined, context: this._context };
this.notify(args);
return args.view;
}
}