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

Hack to respect safe areas #34

Merged
merged 1 commit into from
Nov 13, 2022
Merged

Hack to respect safe areas #34

merged 1 commit into from
Nov 13, 2022

Conversation

AswinPG
Copy link
Collaborator

@AswinPG AswinPG commented Nov 3, 2022

The LayoutChildren method does not work. This is a little hack to make the popup render inside safe area

@LuckyDucko LuckyDucko merged commit c184060 into LuckyDucko:master Nov 13, 2022
@tranb3r
Copy link

tranb3r commented Dec 5, 2022

It looks like HasSystemPadding is now ignored and SystemPadding is always used.
Is HasSystemPadding obsolete ?

@LuckyDucko
Copy link
Owner

For the time being yes, this will be eventually reverted to be fixed however

@AswinPG
Copy link
Collaborator Author

AswinPG commented Dec 5, 2022

I never actually used HasSystemPadding property before, so I didn't think of adding it. Simply adding an if condition bfore applying the padding can make it work. I'll add a PR
@tranb3r @LuckyDucko

@AswinPG
Copy link
Collaborator Author

AswinPG commented Dec 5, 2022

I am really sorry @LuckyDucko I made a mistake and merged the changes instead of creating a PR :|
Didn't know that I had write access..
Sorryyy... What should be done now ..?

@LuckyDucko
Copy link
Owner

@AswinPG thats all good, no worries ahaha, i am the only one able to release a nuget. Give me a litte time to see whats changed

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

Successfully merging this pull request may close these issues.

3 participants