Skip to content

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

Merged
mlynch merged 1 commit intoionic-team:masterfrom
jtomaszewski:patch-1
Dec 6, 2015
Merged

Fix e.detail.keyboardHeight undefined error in keyboardAttach directive#4357
mlynch merged 1 commit intoionic-team:masterfrom
jtomaszewski:patch-1

Conversation

@jtomaszewski
Copy link
Copy Markdown

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
Copy Markdown

Bump, I have this same bug.

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

@jtomaszewski
Copy link
Copy Markdown
Author

I've no idea.

@mlynch mlynch added this to the 1.2 milestone Dec 6, 2015
@mlynch
Copy link
Copy Markdown
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
Copy Markdown
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