Coder Social home page Coder Social logo

Comments (10)

JozefLipovsky avatar JozefLipovsky commented on May 24, 2024

seems to be related to this one #41, make sure you are using latest version.... I have just tested this in my app and it's working fine when VC is embeded in UINavigationController

from lnpopupcontroller.

funkenstrahlen avatar funkenstrahlen commented on May 24, 2024

I am using the latest version. This did not fix it.

However my view hierarchy looks like this, which might be the problem why the fix does not work for me.

popup presents: UIViewController -> UIPageViewController as subview -> UINavigationController -> UIViewController

from lnpopupcontroller.

LeoNatan avatar LeoNatan commented on May 24, 2024

From the screenshots above, I see this is not the latest version. Please update to latest version.

from lnpopupcontroller.

funkenstrahlen avatar funkenstrahlen commented on May 24, 2024

I am currently trying to figure this out

➜  xenim-ios-app git:(feature/chat) carthage update
*** Fetching LNPopupController
*** Checking out LNPopupController at "v1.3"
*** xcodebuild output can be found in /var/folders/8r/nl9r6g3s2j36ljrhdnky9ng40000gn/T/carthage-xcodebuild.Np529A.log
*** Building scheme "LNPopupController" in LNPopupController.xcodeproj

Carthage says I have the current version.

However it is really hard to find out if the code actually has this version, because there is no version number fixed in any files of the repository to crosscheck.

I only found something in Info.plist, but it is outdated: https://github.com/LeoNatan/LNPopupController/blob/master/LNPopupController/Info.plist#L10

from lnpopupcontroller.

funkenstrahlen avatar funkenstrahlen commented on May 24, 2024

I just took a look right in the code and can confirm that the version carthage did checkout actually includes this patch: ac6d3f9

So I assume carthage did build 1.3 correctly.

How can you see my build uses an older version?

from lnpopupcontroller.

LeoNatan avatar LeoNatan commented on May 24, 2024

Hmm, because the popup dismiss button is on top of the bar. But I see you are embedding a navigation controller in a view controller. This all might be iOS fault. Please move the content controller to be a navigation controller directly.

from lnpopupcontroller.

funkenstrahlen avatar funkenstrahlen commented on May 24, 2024

Unfortunately I can not do that because the UINavigationController is just one of my views inside a UIPageViewController. So the popup content controller has to be either the UIPageViewController directly or a UIViewController containing it.

Would also be nice if you could update the version string in Info.plist

from lnpopupcontroller.

funkenstrahlen avatar funkenstrahlen commented on May 24, 2024

If I swipe the UIPageViewController left and right, the UINavigationBar is displayed correctly. It just breaks if the popup is minified. I assume Apples code does not handle the additional top space correctly.

from lnpopupcontroller.

LeoNatan avatar LeoNatan commented on May 24, 2024

That's what I fear. There are many issues with embedding navigation controllers that just don't work well. Unfortunately there is little I can do. Will take a look but will likely close this as an Apple bug.

from lnpopupcontroller.

funkenstrahlen avatar funkenstrahlen commented on May 24, 2024

That's what I fear. There are many issues with embedding navigation controllers that just don't work well. Unfortunately there is little I can do. Will take a look but will likely close this as an Apple bug.

I understand.

This is how it behaves:

wb20jkadax

from lnpopupcontroller.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.