damus icon indicating copy to clipboard operation
damus copied to clipboard

iOS 18: clicking profile picture sometime does nothing

Open jb55 opened this issue 1 year ago • 2 comments

Noticing this sometimes, its likey an iOS 18 issue

jb55 avatar Aug 07 '24 20:08 jb55

It seems this happens if there is no profile or no lightning address ?

jb55 avatar Aug 09 '24 15:08 jb55

I may have experienced this on iOS 17

alltheseas avatar Aug 09 '24 16:08 alltheseas

This is still an issue on the official (non-beta) iOS 18.0 release

Device: iPhone 13 mini (physical device) iOS: 18.0 Damus: Current tip of 1.10 dfa72fceb10dc0cd6867870acfc1f2b69c890722

danieldaquino avatar Sep 18 '24 19:09 danieldaquino

I've asked previously if anyone experiences a similar cannot tape on share note externally icon (bottom right of any note)

alltheseas avatar Sep 19 '24 09:09 alltheseas

Looking into this

danieldaquino avatar Sep 20 '24 18:09 danieldaquino

I traced back the issue to KFAnimatedImage. When I replace that with SwiftUI's AsyncImage the tap gesture is recognized. It seems like that view is not capturing tap gestures on iOS 18.

Very likely https://github.com/damus-io/damus/issues/2342 has the exact same root cause and fix

danieldaquino avatar Sep 20 '24 18:09 danieldaquino

I tried updating Kingfisher (the library that defines KFAnimatedImage) to see if they have fixed this on their end:

  • The latest version of Kingfisher (8.0.2) is not compatible with our implementation (I won't try to fix compatibility issues yet to save time for the release)
  • The latest 7.x Kingfisher version (7.12.0) also has the same issue.

I will try implementing a quick solution on our end that should fix the issue while we don't fix this on the Kingfisher side

danieldaquino avatar Sep 20 '24 18:09 danieldaquino

Implemented https://github.com/damus-io/damus/pull/2498 as a tentative PR which improves the iOS 18 situation. Not a final fix, but addresses this if we can't get a proper fix in on time.

danieldaquino avatar Sep 20 '24 20:09 danieldaquino

I will keep investigating to find a better fix.

danieldaquino avatar Sep 20 '24 20:09 danieldaquino

I created a minimal reproducible Xcode project that shows the issue is present on the latest Kingfisher version too (8.0.2).

Not actively debugging this right now, I am focusing on https://github.com/damus-io/damus/issues/2458

danieldaquino avatar Sep 20 '24 22:09 danieldaquino