Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix e.detail.keyboardHeight undefined error in keyboardAttach directive #4357

Merged
merged 1 commit into from
Dec 6, 2015

Conversation

jtomaszewski
Copy link

My background:

  • newest ionic
  • newest ionic cordova keyboard plugin installed
  • iOS 8 + iPhone 5C
  • debugging through Safari Emulator

What happens:

When focusing on the input in <ion-footer-bar keyboard-attach>, I get a TypeError: undefined is not an object (evaluating 'e.detail.keyboardHeight') error.

This bugfix fixes it.

…tive

My background:

- newest ionic
- newest ionic cordova keyboard plugin installed
- iOS 8 + iPhone 5C
- debugging through Safari Emulator

What happens:

When focusing on the input in `<ion-footer-bar keyboard-attach>`, I get a `TypeError: undefined is not an object (evaluating 'e.detail.keyboardHeight')` error.

This bugfix fixes it.
@jtomaszewski jtomaszewski changed the title Fix e.detail.keyboardHeight undefined error in keyboardAttach direc… Fix e.detail.keyboardHeight undefined error in keyboardAttach directive Sep 14, 2015
@joshdover
Copy link

Bump, I have this same bug.

@jtomaszewski do you know what is actually causing this event to not have detail?

@jtomaszewski
Copy link
Author

I've no idea.

@mlynch mlynch added this to the 1.2 milestone Dec 6, 2015
@mlynch
Copy link
Contributor

mlynch commented Dec 6, 2015

Tracking for 1.2, thanks

mlynch added a commit that referenced this pull request Dec 6, 2015
Fix `e.detail.keyboardHeight` undefined error in keyboardAttach directive
@mlynch mlynch merged commit d884ecb into ionic-team:master Dec 6, 2015
@mlynch
Copy link
Contributor

mlynch commented Dec 6, 2015

Err, one liner easy to merge 😄 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants