search

SAP / openui5

OpenUI5 lets you build enterprise-ready web applications, responsive to all devices, running on almost any browser of your choice.
http://openui5.org
Apache License 2.0
2.96k stars 1.24k forks source link

UI5 Pull-to-Refresh no longer works in Native Android App. #3373

Closed webbpage closed 2 years ago

webbpage commented 3 years ago

OpenUI5 version: 1.87

Browser/version (+device/version): Android 7: LGE LGL83BL Android 10: LENOVO YT-X705F

Any other tested browsers/devices(OK/FAIL): iOS 14.5.1: iPhone 11 (Works)

URL (minimal example if possible): https://docs.google.com/document/d/1n3HtvpXRhwOFSky2GdZjfEQ5pKnlNvUA467kFH03hho/edit?usp=sharing

User/password (if required and possible - do not post any confidential information here):

Steps to reproduce the problem:

  1. Download SAP sample pull-to-refresh app
  2. Deploy as native Android App (not web app)
  3. Test pull-to-refresh - nothing happens

What is the expected result? Pull to refresh on Android would work the same as SAPUI5 demo app https://openui5.hana.ondemand.com/entity/sap.m.PullToRefresh/sample/sap.m.sample.PullToRefresh

What happens instead? Nothing

Any other information? (attach screenshot if possible): see google doc

nnaydenow commented 3 years ago

Hi @webbpage,

Which browser you are using on those devices? It works okey to me on my iOS device (Iphone 11 pro) with safari browser and on my android device (Huawei P20) with chrome browser.

Regards, Nayden

webbpage commented 3 years ago

I'm using the Android and iOS WebViews. You have to deploy to Android as a Web Native app to test this issue.

Android Webview 95.0.4638.50 https://developer.android.com/reference/android/webkit/WebView

While debugging I noticed that many of SAPUI5's 3rd-party library dependencies are a bit out of date (eg. iscroll). I tried reaching out to the iscroll team directly, but did not hear back.

This appears to be related: " iScroll for iOS and it works great for iOS platforms. But none of the methods works for android platform" https://stackoverflow.com/questions/21709737/pull-to-refresh-for-android-and-phonegap

Thanks

On Fri, Oct 22, 2021 at 7:00 AM Nayden Naydenov @.***> wrote:

Hi @webbpage https://github.com/webbpage,

Which browser you are using on those devices? It works okey to me on my iOS device (Iphone 11 pro) with safari browser and on my android device (Huawei P20) with chrome browser.

Regards, Nayden

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/SAP/openui5/issues/3373#issuecomment-949519266, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD7ZVJBCIC635BYGNIFFNTUIE74LANCNFSM5GOKX73A .

webbpage commented 2 years ago

I'm happy to see that the reported Android Pull-to-Refresh issue is "in progress". Please let me know if there's anything I can do to help gain progress.

flovogt commented 2 years ago

Hello @webbpage , Thank you for sharing this finding. I've created an internal incident 2180423465. The status of the issue will be updated here in GitHub. Regards, Florian

webbpage commented 2 years ago

Thank you for the update @flovogt

webbpage commented 2 years ago

@flovogt @HerzogIgzorn Any update for this issue? I reviewed the recent 1.97.1 release notes, but I did not see mention of a pull-to-refresh fix for native Android. Thanks again.

jdichev commented 2 years ago

Hello and sorry for delay, this was brought to our team QE and quoting the response:

Regarding our support Matrix we are supporting WebView only on Windows and iOS. This means that we are not supporting WebView with Android which is described in GitHub scenario. For more details you could check here: https://sapui5.hana.ondemand.com/sdk/#/topic/74b59efa0eef48988d3b716bd0ecc933

If the issue is not reproducible in a typical Android browser like Chrome, this could be attributed to the WebView itself or a configuration on the application related to scrolling.

I hope this helps.

Best regards, Jordan