javascript/ionic-framework
1
0
Fork 0
mirror of https://github.com/ionic-team/ionic-framework.git synced 2025-08-19 11:41:20 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(menu): default menu mode

Browse Source
This commit is contained in:
Manu Mtz.-Almeida
2018-03-27 23:07:56 +02:00
parent 5007c78a40
commit c31bcde720
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
core/src/components/menu/menu.tsx
View File

@ -52,7 +52,7 @@ export class Menu {
* see the `menuType` in the [config](../../config/Config). Available options:
* `"overlay"`, `"reveal"`, `"push"`.
*/
@Prop({ mutable: true }) type = 'overlay';
@Prop({ mutable: true }) type: string;
@Watch('type')
typeChanged(type: string, oldType: string | null) {
@ -122,6 +122,9 @@ export class Menu {
async componentWillLoad() {
this.menuCtrl = await this.lazyMenuCtrl.componentOnReady();
if (this.type == null) {
this.type = this.mode === 'ios' ? 'reveal' : 'overlay';
}
}
componentDidLoad() {
@ -232,7 +235,7 @@ export class Menu {
// Menu swipe animation takes the menu's inner width as parameter,
// If `offsetWidth` changes, we need to create a new animation.
const width = this.menuInnerEl.offsetWidth;
if (width === this.width && this.animation !== null) {
if (width === this.width && this.animation !== undefined) {
return;
}
this.width = width;
@ -240,7 +243,7 @@ export class Menu {
// Destroy existing animation
if (this.animation) {
this.animation.destroy();
this.animation = null;
this.animation = undefined;
}
// Create new animation
this.animation = await this.menuCtrl.createAnimation(this.type, this);