Resolved Issue NativeScript#1127: Changing ActionBar title after page is loaded does not show the ActionBar

hamorphis · hamorphis · commit b47a648e6cfb · 2015-12-14T15:58:17.000+02:00
diff --git a/ui/action-bar/action-bar.ios.ts b/ui/action-bar/action-bar.ios.ts
@@ -169,6 +169,10 @@ export class ActionBar extends common.ActionBar {
             return;
         }
 
+        if (this.page.frame) {
+            this.page.frame._updateActionBar();
+        }
+
         var navigationItem: UINavigationItem = (<UIViewController>this.page.ios).navigationItem;
         navigationItem.title = this.title;
     }
diff --git a/ui/frame/frame.ios.ts b/ui/frame/frame.ios.ts
@@ -420,9 +420,9 @@ class iOSFrame implements definition.iOSFrame {
         this._showNavigationBar = value;
         this._controller.navigationBarHidden = !value;
 
-        let owner = this._controller.owner;
-        if (owner && change) {
-            owner.requestLayout();
+        let currentPage = this._controller.owner.currentPage;
+        if (currentPage && change) {
+            currentPage.requestLayout();
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -169,6 +169,10 @@ export class ActionBar extends common.ActionBar {`
`169`	`169`	`return;`
`170`	`170`	`}`
`171`	`171`
	`172`	`+ if (this.page.frame) {`
	`173`	`+ this.page.frame._updateActionBar();`
	`174`	`+ }`
	`175`	`+`
`172`	`176`	`var navigationItem: UINavigationItem = (<UIViewController>this.page.ios).navigationItem;`
`173`	`177`	`navigationItem.title = this.title;`
`174`	`178`	`}`
Original file line number	Diff line number	Diff line change
`@@ -420,9 +420,9 @@ class iOSFrame implements definition.iOSFrame {`
`420`	`420`	`this._showNavigationBar = value;`
`421`	`421`	`this._controller.navigationBarHidden = !value;`
`422`	`422`
`423`		`- let owner = this._controller.owner;`
`424`		`- if (owner && change) {`
`425`		`- owner.requestLayout();`
	`423`	`+ let currentPage = this._controller.owner.currentPage;`
	`424`	`+ if (currentPage && change) {`
	`425`	`+ currentPage.requestLayout();`
`426`	`426`	`}`
`427`	`427`	`}`
`428`	`428`