Skip to content

Commit

Permalink
fix(overlay): overlay is not hidden
Browse files Browse the repository at this point in the history
  • Loading branch information
manucorporat committed Aug 29, 2018
1 parent 2268346 commit 89ba55d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
4 changes: 1 addition & 3 deletions core/src/components/datetime/datetime.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -232,9 +232,7 @@ export class Datetime {
const pickerOptions = this.generatePickerOptions();
this.picker = await this.pickerCtrl.create(pickerOptions);
await this.validate();
if (this.picker) {
await this.picker.present();
}
await this.picker.present();
}

private emitStyle() {
Expand Down
4 changes: 2 additions & 2 deletions core/src/utils/overlays.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export function createOverlay<T extends HTMLIonOverlayElement>(element: T, opts:
// convert the passed in overlay options into props
// that get passed down into the new overlay
Object.assign(element, opts);
element.classList.add('ion-page-hidden');
element.classList.add('ion-page-invisible');
const overlayIndex = lastId++;
element.overlayIndex = overlayIndex;
if (!element.hasAttribute('id')) {
Expand Down Expand Up @@ -137,7 +137,7 @@ async function overlayAnimation(
}

// Make overlay visible in case it's hidden
baseEl.classList.remove('ion-page-hidden');
baseEl.classList.remove('ion-page-invisible');

const aniRoot = baseEl.shadowRoot || overlay.el;
const animation = overlay.animation = await overlay.animationCtrl.create(animationBuilder, aniRoot, opts);
Expand Down

0 comments on commit 89ba55d

Please sign in to comment.