ably / ably-asset-tracking-swift

iOS client SDKs for the Ably Asset Tracking service.
Apache License 2.0
9 stars 6 forks source link

Publisher example app: Understand why "Remove trackable" button isn't tappable without adding some space above it #422

Open lawrence-forooghian opened 1 year ago

lawrence-forooghian commented 1 year ago

For some reason, the "Remove trackable" button on the publisher example map view wouldn't respond to my taps. I couldn't understand why. I added some vertical whitespace above the button, and it started working. I don't understand why. It would be good to understand the original issue and fix it.

JakubJankowski commented 1 year ago

I've run some tests - I couldn't reproduce this problem after removing the padding. Ran on a couple of simulators (iPhone 13 Pro Max, iPhone 13, iPhone SE 3rd gen) and on a physical 6s Plus. Maybe it was accidentally fixed somewhere along the way? Or maybe it happens only on some specific configuration that you used and I couldn't replicate.

Either way - I won't delete this padding or close this issue yet. @lawrence-forooghian please double check if this still happens on your setup when you find some time.

lawrence-forooghian commented 1 year ago

Do you know what OS versions you ran on, @JakubJankowski?

JakubJankowski commented 1 year ago

As mentioned on a call - 15.6 on all devices. I'll update my physical device tomorrow to 16.x and I'll try one more time.

JakubJankowski commented 1 year ago

Whoops, looks like Apple dropped support for my good old 6s plus. Can't update it to 16.x I'm afraid. And 16.x simulators are unavailable on Xcode 13.x, so I can't test on that right now either.

deanna-lad commented 1 year ago

The corresponding Jira issue is https://ably.atlassian.net/browse/SDK-3291