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

Patch TabNavigation and AppBar so they render within the viewport in iOS #114

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

enzomonjardin
Copy link
Contributor

#Problem
Certain components are rendering with some issues on iOS

  • AppBars still have white padding on top
  • Bottom TabNavigation almost completely out of viewport

#Solution
Patches components with issues for iOS

#Notes

The changes to TabNavigation might not be needed after merging https://github.com/AdaloHQ/backend/pull/946 , since it addresses some issues when calculating fixed element positions.

@enzomonjardin enzomonjardin requested review from jose-adalo and a team February 6, 2023 16:54
@enzomonjardin
Copy link
Contributor Author

@enzomonjardin enzomonjardin force-pushed the enzo-cb02-fix-ios-rwa-issues branch from 6ed0ee6 to f0651f3 Compare February 7, 2023 17:53
@enzomonjardin
Copy link
Contributor Author

Updated this PR, set the style changes to only apply if the components are part of a responsive web app.
This depends on runner/pull/680 to work, since it adds the isResponsiveComponent prop to LibraryComponents

@jose-adalo
Copy link

@enzomonjardin can we remove the TabBar Navigation changes

Copy link

@jose-adalo jose-adalo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the Tab Navigation changes

src/TabNavigator/index.js Show resolved Hide resolved
Copy link

@jose-adalo jose-adalo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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.

2 participants