mobify / pinny

A mobile-first content fly-in UI plugin
MIT License
23 stars 4 forks source link

Fix 63: Pinny gets distorted when the soft keyboard is shown after orientationchange #66

Closed hcstephencheung closed 9 years ago

hcstephencheung commented 9 years ago

Status: Opened for visibility JIRA: https://mobify.atlassian.net/browse/CSOPS-1049 Reviewers: @scalvert @Helen-Mobify

Changes

hcstephencheung commented 9 years ago

@scalvert @Helen-Mobify @ericmuyser Please review.

hcstephencheung commented 9 years ago

@scalvert @Helen-Mobify @ericmuyser Please review.

wizardlyhel commented 9 years ago

@hcstephencheung you can save the reference to 'this' before the orientationchange function

hcstephencheung commented 9 years ago

@Helen-Mobify Done. @scalvert @ericmuyser

hcstephencheung commented 9 years ago

@Helen-Mobify : thanks, changed _this to plugin @scalvert : I made the util function _isWithin. Let me know if you want it to return something else other than just true or false. (maybe just return the length?)

kpeatt commented 9 years ago

This seems good — should we make a test for it?

scalvert commented 9 years ago

@kpeatt good to go!

scalvert commented 9 years ago

To reply to your comment re: testing, I don't think we can devise a decent test for this. So no?