Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ContentView left inset of -8 #551

Closed
WouterVermeijNPO opened this issue Nov 24, 2023 · 8 comments
Closed

ContentView left inset of -8 #551

WouterVermeijNPO opened this issue Nov 24, 2023 · 8 comments
Labels

Comments

@WouterVermeijNPO
Copy link

WouterVermeijNPO commented Nov 24, 2023

Describe the Bug
The _LNPopupBarContentView leading inset is suddenly inset out of view.

Our project was on version 2.15.4 and i updated to 2.15.24 and the issue arose. I pinpointed it in the 2.15.22 release where a left inset is set on the contentFrame of -8.

I think this is the line affecting us

2.15.21...2.15.22#diff-4f6881108fb36c4b054ad3dba6575391825d8cf796acfb1b62e2482ab309364dR554

I updated to the latest version, 2.16.4, to check if it was fixed there, but it wasn't.

Expected Behavior
I don't expect the view to change insets on an update 😛

Screenshots
Screenshot 2023-11-24 at 16 01 03

Screenshot 2023-11-24 at 16 03 09

Additional Context
I really wonder why you decided to add this inset. Next to that, what would a proper solution be for our project?

@LeoNatan
Copy link
Owner

Hello,
Is this a custom popup bar?
I'll try to fix the issue as soon as possible.

While this is a bug, the reason why this was needed is for other layouts, and I guess I didn't properly gate it to those situations.

@WouterVermeijNPO
Copy link
Author

WouterVermeijNPO commented Nov 24, 2023

Hey, thanks for the quick reply!

We are indeed using a customBarViewController

@LeoNatan
Copy link
Owner

LeoNatan commented Nov 24, 2023

Just an aside, but any reason you are not using the standard prominent style? You should be able to achieve your current look. You can customize font, colors, etc.

@WouterVermeijNPO
Copy link
Author

Not that i know of, i inherited the project unfortunately. But it might be worth looking into now that you mention it.

@LeoNatan
Copy link
Owner

Please try with the 2.16.5 release.

@LeoNatan
Copy link
Owner

I do suggest using the standard layout styles as much as possible.

https://github.com/LeoNatan/LNPopupController#bar-style

In addition to getting a lot of behavior "for free", they are also the most maintained, and are very customizable, so you can achieve a lot with them.

@WouterVermeijNPO
Copy link
Author

Amazing man, thanks a lot for the quick response and fix!

@LeoNatan
Copy link
Owner

Cheers! Happy coding!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants