EZCustomNavigation icon indicating copy to clipboard operation
EZCustomNavigation copied to clipboard

Workaround legal swipe actions from the left side

Open ReDetection opened this issue 4 years ago • 3 comments
trafficstars

How I was able to fix #6

ReDetection avatar Jan 13 '21 08:01 ReDetection

I just noticed that proposed solution might be not optimal. What I did was to require all other gesture recognizers to fail (basically force-included all gestures to fall into the trap and opt out scroll-view one). Maybe we should instead only require UITableView recognizers to fail in order for EZ pan recognizer to recognize (so the other way around – none are required to fail except what we want to work). I have two reasons in mind:

  1. In my app I suddenly start to conflict with UXCam pod (basically a visor) – they seem to install global pinch gesture recognizer which also doesn't fail quickly.
  2. Future iOS updates might add more gesture recognizers and I find "opt-in" way safer

I will include my updated solution here and push a new commit, but sure it's up to you which way to choose.

ReDetection avatar Jan 13 '21 11:01 ReDetection

I'm not sure this is the proper way forward. Need to further investigate, but probably we're gonna need to defer some of this responsibility to the app which actually knows what kind of gesture it uses rather than we having to guess for it.

Enricoza avatar Jan 13 '21 22:01 Enricoza

Sounds reasonable. Maybe a closure in the EZNavigationConfiguration? Please let me know if you have a suggestion or feel free to close the PR if your own way :)

ReDetection avatar Jan 14 '21 02:01 ReDetection