klarna / klarna-mobile-sdk

Klarna Mobile SDK for iOS
https://docs.klarna.com/mobile-sdk/ios/
Apache License 2.0
29 stars 12 forks source link

Can not show Staging View after using 2.0.44 version SDK #113

Closed hanorz closed 3 years ago

hanorz commented 3 years ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Device and version:

Merchant Name Please state the merchant's name and the application's name that is facing the issue using the SDK.

Impact Level What is the impact level of this bug on the merchant and its customers? Is it happening in production? Is it a blocking issue for development? etc.

Additional context

After using 2.0.44 version SDK, the tap action of the KlarnaOSMView was disable. Which means it can not show the Staging view. And I tried many times figured out the KlarnaOSMView only respond the small area in the picture.

WeChatd88f540cd9c6c1481e3f45eaffa7dd18

polar133 commented 3 years ago

Hi, @hanorz can you please share the device and OS that you are using for testing? On our end, the label with the underline works correctly to detect the tap gesture that opens the fullscreen view.

hanorz commented 3 years ago

Hi, @hanorz can you please share the device and OS that you are using for testing? On our end, the label with the underline works correctly to detect the tap gesture that opens the fullscreen view.

I am using iPhone 12 simulator(14.5) and iPhone 8 real device(14.4).

hanorz commented 3 years ago

Hi, @hanorz can you please share the device and OS that you are using for testing? On our end, the label with the underline works correctly to detect the tap gesture that opens the fullscreen view.

The screenshot picture was build in iPhone 12 simulator(14.5), there is nothing else on the project, only 2.0.44 SDK

polar133 commented 3 years ago

Hi, @hanorz can you please share the device and OS that you are using for testing? On our end, the label with the underline works correctly to detect the tap gesture that opens the fullscreen view.

The screenshot picture was build in iPhone 12 simulator(14.5), there is nothing else on the project, only 2.0.44 SDK

Thanks for the info. We will try to replicate the same issue. Meanwhile, can you try the OSM with the environment = .demo, it should render a static osm view, please, check if that view has the same issue. tap in any part of the underline and should open a fullscreen. Let us know if that one works please.

hanorz commented 3 years ago

Hi, @hanorz can you please share the device and OS that you are using for testing? On our end, the label with the underline works correctly to detect the tap gesture that opens the fullscreen view.

The screenshot picture was build in iPhone 12 simulator(14.5), there is nothing else on the project, only 2.0.44 SDK

Thanks for the info. We will try to replicate the same issue. Meanwhile, can you try the OSM with the environment = .demo, it should render a static osm view, please, check if that view has the same issue. tap in any part of the underline and should open a fullscreen. Let us know if that one works please.

Just tried. Same issue, nothing react by tap.

hanorz commented 3 years ago

Only react by this small area. 企业微信截图_6aaa55f2-07ab-4a2e-9712-e2b1bd29a8d4

polar133 commented 3 years ago

@hanorz thanks. Could you share how the Debug View Hierarchy looks for that element please ? also, which version of xcode ?

hanorz commented 3 years ago

@hanorz thanks. Could you share how the Debug View Hierarchy looks for that element please ? also, which version of xcode ?

WeChatcab847a3e3d9eb1b2d3e0228c05dae23

WeChat5b2c06dda2ebcd5ceb7ffb2eaaa94f98

WeChat5895b24ec36c2e6a47b1e2de3fbccb04

Version 12.5.1 for Xcode

hanorz commented 3 years ago

@hanorz thanks. Could you share how the Debug View Hierarchy looks for that element please ? also, which version of xcode ?

I can send you the demo project if you need.

polar133 commented 3 years ago

@hanorz thanks for the info. It looks like it's a height constraint on the OSM view in the implementation. Make sure to extend KlarnaOSMViewEventListener and use the height sent in klarnaOSMViewResized, and update the height constraint for the OSM view with the new height sent.

132853045-51c05bed-4ce8-4b2f-9dee-3b369e223425

hanorz commented 3 years ago

@hanorz thanks for the info. It looks like it's a height constraint on the OSM view in the implementation. Make sure to extend KlarnaOSMViewEventListener and use the height sent in klarnaOSMViewResized, and update the height constraint for the OSM view with the new height sent.

132853045-51c05bed-4ce8-4b2f-9dee-3b369e223425

hanorz commented 3 years ago

It works.Thanks you.