diff --git a/apps/tests/ui/page/page-tests-common.ts b/apps/tests/ui/page/page-tests-common.ts index 6daf6eac7..a9f2cb0d1 100644 --- a/apps/tests/ui/page/page-tests-common.ts +++ b/apps/tests/ui/page/page-tests-common.ts @@ -27,6 +27,7 @@ import LabelModule = require("ui/label"); import stackLayoutModule = require("ui/layouts/stack-layout"); import helper = require("../helper"); import view = require("ui/core/view"); +import builder = require("ui/builder"); export function addLabelToPage(page: PageModule.Page, text?: string) { var label = new LabelModule.Label(); @@ -68,6 +69,15 @@ export function test_menuItem_inherit_bindingContext() { } } +export function test_menuItem_inherit_bindingContext_inXML() { + var p = builder.parse(""); + p.bindingContext = { myProp: "success" }; + + var menuItem = p.optionsMenu.getItemAt(0); + + TKUnit.assertEqual(menuItem.text, "success", "menuItem.text"); +}; + export function test_Setting_OptionsMenu_doesnt_thrown() { var page: PageModule.Page; @@ -392,4 +402,4 @@ export var test_cssShouldBeAppliedAfterChangeToAllNestedElements = function () { finally { helper.goBack(); } -} +} \ No newline at end of file diff --git a/ui/page/page-common.ts b/ui/page/page-common.ts index e21aef45a..bacc95990 100644 --- a/ui/page/page-common.ts +++ b/ui/page/page-common.ts @@ -206,7 +206,20 @@ export class OptionsMenu implements dts.OptionsMenu { } public setItems(items: Array) { - this._items = items; + if (this._items === items) { + return; + } + + // Remove all existing items + while (this._items.length > 0) { + this.removeItem(this._items[this._items.length - 1]); + } + + // Add new items + for (var i = 0; i < items.length; i++) { + this.addItem(items[i]); + } + this.invalidate(); }