Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

cordova.plugins.Keyboard.hideKeyboardAccessoryBar(true); is no work #85

Closed
ckken opened this issue Mar 16, 2015 · 27 comments
Closed

cordova.plugins.Keyboard.hideKeyboardAccessoryBar(true); is no work #85

ckken opened this issue Mar 16, 2015 · 27 comments

Comments

@ckken
Copy link

ckken commented Mar 16, 2015

cordova.plugins.Keyboard.hideKeyboardAccessoryBar(true);

@admmasters
Copy link

Seconded - this is pretty key.

@hussfelt
Copy link

Related #27

@hussfelt
Copy link

We are seeing the same issue

@fungilation
Copy link

+1. A key plugin we are relying on for consistent mobile UI.

@Wenape
Copy link

Wenape commented Jun 1, 2015

+1

@etiennea
Copy link

etiennea commented Jun 3, 2015

+1 :-/

@scripterkaran
Copy link

+1

7 similar comments
@clauderic
Copy link

+1

@enricodeleo
Copy link

+1

@aemr3
Copy link

aemr3 commented Jun 21, 2015

+1

@rumit91
Copy link
Contributor

rumit91 commented Jun 23, 2015

+1

@itkin
Copy link

itkin commented Jun 28, 2015

+1

@1nsaneinc
Copy link

+1

@woniesong92
Copy link

+1

@hussfelt
Copy link

Should this issue be closed now that #27 is fixed?

ping @EddyVerbruggen

@EddyVerbruggen
Copy link
Contributor

@hussfelt If you can confirm that the problem in this issue is fixed then please feel free to do so.

@hussfelt
Copy link

@EddyVerbruggen will try and confirm this when we do another sprint-test with WKWebView and ping you to close. I can't as I did not create the issue :)

@clauderic
Copy link

I can't confirm regarding the ionic keyboard plugin, but as for cordova-plugin-keyboard, Keyboard.hideFormAccessoryBar(true) still does not work

@chriswessels
Copy link

+1. We really need this.

@andreamaioli
Copy link

+1

1 similar comment
@berndartmueller
Copy link

+1

@andreialecu
Copy link
Contributor

For anyone reading this, it has been fixed recently in cjpearson/cordova-plugin-keyboard@33cbbd9

@hussfelt
Copy link

This is pure gold!

@cjpearson
Copy link

To add on to the comment from @andreialecu, that change has now been published to npm in 1.1.3.

@EddyVerbruggen
Copy link
Contributor

Thanks @cjpearson - adding a reference to https://www.npmjs.com/package/cordova-plugin-keyboard-ams to the readme.

@cjpearson
Copy link

Thanks @EddyVerbruggen, that's a different fork though. The one with the fix is here: https://www.npmjs.com/package/cordova-plugin-keyboard. Also, it looks like #187 may duplicate this.

@EddyVerbruggen
Copy link
Contributor

Thx, doc fixed.

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

No branches or pull requests