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

Fixes 1191767 - Do not manually handle keyboard overlapping with WKWebView #877

Merged
merged 1 commit into from Aug 7, 2015

Conversation

st3fan
Copy link
Contributor

@st3fan st3fan commented Aug 6, 2015

This patch removes all keyboard handling code and lets the WKWebView handle it.

@@ -90,15 +90,22 @@ class SettingsContentViewController: UIViewController, WKNavigationDelegate {

self.webView = makeWebView()
view.addSubview(webView)
self.webView.snp_remakeConstraints { make in
make.left.right.top.equalTo(self.view)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any reason why this can't be make.left.right.top.bottom.equalTo(self.view)?

or even make.edges.equalTo(self.view)

@st3fan st3fan force-pushed the st3fan/Bug1191767LoginWebViewScrolling branch from 7642b76 to 35302d2 Compare August 7, 2015 13:08
st3fan added a commit that referenced this pull request Aug 7, 2015
…olling

Fixes 1191767 - Do not manually handle keyboard overlapping with WKWebView
@st3fan st3fan merged commit 5e9feda into master Aug 7, 2015
@farhanpatel farhanpatel deleted the st3fan/Bug1191767LoginWebViewScrolling branch October 6, 2016 07:01
isabelrios pushed a commit to isabelrios/firefox-ios that referenced this pull request Feb 19, 2024
* Updating existing locales: br, ca, da, dsb, en, es-AR, eu, gd, he, hi-IN, hsb, it, ka, kk, lo, nl, nn, pl, pt-BR, ro, sq, ta, th, tr, zh-CN
* Importing new locales: an, ia, hy-AM
* Include all locales in screenshots.sh
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.

None yet

2 participants