javascript/NativeScript
1
0
Fork 0
mirror of https://github.com/NativeScript/NativeScript.git synced 2025-08-15 19:26:42 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: dont create an actionbar if not necessary (#8402)

Browse Source 
* fix: dont create an actionbar if not necessary

For now i kept the commented code so that you can see the change.
Also i changed the android check to behave like iOS

* rollback. we now try an make sure the actionbar  is created only if needed

* actually we should check for false
This commit is contained in:
Martin Guillon
2020-03-19 10:00:29 +01:00
committed by GitHub
parent 89ee60be00
commit 4a67a3b73f
2 changed files with 17 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
nativescript-core/ui/page/page-common.ts
View File

@ -138,7 +138,7 @@ export class PageBase extends ContentView implements PageDefinition {
public eachChildView(callback: (child: View) => boolean) {
super.eachChildView(callback);
if (this.actionBar) {
if (this.hasActionBar) {
callback(this.actionBar);
}
}

19
nativescript-core/ui/page/page.android.ts
View File

@ -45,17 +45,30 @@ export class Page extends PageBase {
@profile
public onLoaded() {
super.onLoaded();
if (this.actionBarHidden !== undefined) {
if (!this.hasActionBar && this.actionBarHidden !== true) {
// ensure actionBar is created
// but we only need to do that if the actionBarHidden is not hidden
this.actionBar = new ActionBar();
}
if (this.hasActionBar) {
this.updateActionBar();
}
}
private updateActionBar() {
this.actionBar.update();
// the test is actually to ensure the actionBar is created
// it will be created if not
if (this.actionBar) {
this.actionBar.update();
}
}
[actionBarHiddenProperty.setNative](value: boolean) {
this.updateActionBar();
// in case the actionBar is not created and actionBarHidden is changed to true
// the actionBar will be created by updateActionBar
if (!value || this.hasActionBar) {
this.updateActionBar();
}
}
[statusBarStyleProperty.getDefault](): { color: number, systemUiVisibility: number } {