fix(5907): webview bouncing issue on ios 16

[rpanadero]

Fixes issue #5907

[jcesarmobile]

Can you provide a video or gif of the behavior before and after the change?
I have tested the code and the WebView is still bouncing with the proposed changes.

[thoasty-dev]

I can confirm the fix works. Would be great if you got this merged.

[kemalcany]

Tested on the simulator and the bouncing seems to be fixed (haven't tested on a physical device tho)

[giralte-ionic]

Could we get some videos or gifs of the problem behavior? We are thinking this may be a 16.0 issue and could be fixed in 16.1, so I'd also recommend testing with the current iOS Beta

[thoasty-dev]

It is not fixed on iOS 16.1 Developer Beta.

Video from a real device (iPhone 14 Pro): https://imgur.com/a/40XFTc0

[fabltd]

How do I install this branch?

[Sharcoux]

@rpanadero Do you use your fork internally? How can we use it?

[rpanadero]

@rpanadero Do you use your fork internally? How can we use it?

@Sharcoux no, I don't. As a temporal fix until this PR is merged/reviewed, I developed an ad-hoc plugin for my project that changes the Capacitor webview configuration. Just add these two files to your iOS project:

WebViewPlugin.m

#import <Capacitor/Capacitor.h>

CAP_PLUGIN(WebViewPlugin, "WebViewPlugin",)

WebViewPlugin.swift

import Capacitor

@objc(WebViewPlugin)
public class WebViewPlugin: CAPPlugin {
    @objc override public func load() {
        // Called when the plugin is first constructed in the bridge
        self.bridge?.webView?.scrollView.bounces = false
        self.bridge?.webView?.scrollView.alwaysBounceVertical = false
        self.bridge?.webView?.scrollView.alwaysBounceHorizontal = false
    }
}

After adding/creating them to your iOS project, be sure that the files are included in the bundle resources.

[IJustDev]

@rpanadero thank you very much this works for me!

If anybody reading this and don't know how to put this in practice:

• open Xcode (VSCode won't track them in the build process)
• create both files respectively on the same level as AppDelegate

[IJustDev]

LGTM!

[willgardner9]

Thanks for the instructions @rpanadero and @IJustDev! I can't seem to get web view bounce to stop. I have created both files in the same dir as the AppDelegate.swift - I'm sure this is my fault for missing something

Do I need to create a bridging file or anything? I can see the Xcode project has changed and the files are present in my ./ios/App/App.xcodeproj/project.pbxproj file

Appreciate all your efforts on this, thanks so much 🙌

[IJustDev]

Do these files appear in XCode?

[willgardner9]

Yep, the files appear as expected in XCode

I also experimented by adding:

  ios: {
    scrollEnabled: false,
  },

to my capacitor.config.ts and that still didn't fix the bounce, so I think perhaps there is some other issue at work not related to the code in this PR. Have run npx cap sync etc, will report back if I find out what's going on 👍

[bsalesc]

The above code didn't work, so creating an object c file with the below code worked for me:

#import <Foundation/Foundation.h>
#import <UIKit/UIKit.h>

@implementation UIScrollView (NoBounce)
- (void)didMoveToWindow {
    [super didMoveToWindow];
    self.bounces = NO;
}
@end

Found this code here: #1169 (comment)

So maybe moving this piece of code inside WKWebView+Capacitor.m might work for everyone?

[piotrfonte]

Can’t get any of the above snippets to work to solve the problem. How can I check if added files get included in the bundle?

[thoasty-dev]

This is not an issue anymore on iOS 16.2

[piotrfonte]

Gotcha. Can confirm that this is fixed in iOS 16.2.

[BenjaminDish]

This is not an issue anymore on iOS 16.2

@thoasty-dev You say it is fixed on ios 16.2 but i can't understand what is the expected behaviour.

Is bouncing automatically disabled in capacitor apps by default ? (i don't think so as bouncing is ON on my ios 14 iphone)

Or, does it need any configuration/code to disable it ? If yes, which one ?

[brilliantinsane]

Will this ever be merged?