-
Notifications
You must be signed in to change notification settings - Fork 3.9k
feat: Add logic for better bottom SafeArea handling + Enable translucent Android NavBar #57128
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
Merged
mountiny
merged 91 commits into
Expensify:main
from
margelo:@chrispader/add-flags-for-disabling-bottom-safe-area-spacing
Mar 14, 2025
Merged
Changes from all commits
Commits
Show all changes
91 commits
Select commit
Hold shift + click to select a range
cd65e37
refactor: rename StyleUtils safe area padding function
chrispader a4e915c
refactor: simplify code in `useStyledSafeAreaInsets` hook
chrispader 0fbfb27
feat: implement android navigation bar type specific bottom safe area…
chrispader 124cc53
fix: add source url
chrispader e602c2f
fix: use CONST values and add comments
chrispader 490e560
update comment about navigation bar height threshold
chrispader e17118e
fix: revert navigation bar changes in `getPlatformSafeAreaPadding`
chrispader 9c23b7c
feat: disable bottom safe area padding handling completely
chrispader cdccefa
fix: syntax and eslint error in ScreenWrapper
chrispader 1bbb3b7
fix: safe area padding in CategorySelectorModal (SelectionList flag)
chrispader ffcf463
fix: missing parantheses -> wrong bottom safe area padding
chrispader f27e1b0
feat: simplify `useStyledSafeAreaInsets` hook
chrispader 26f9d5c
feat: add BaseModal flag for disabling bottom safe area spacing
chrispader 0999a97
fix: unmodifiedPaddings was removed
chrispader 042a464
rename temporary flag for bottom safe area handling
chrispader 5aa0b04
feat: implement `NavBarManager.setBackgroundColor` for Android transl…
chrispader 06c73c6
fix: implement NavigationBar component and use instead of native NavB…
chrispader 56e4fdc
fix: ScreenWrapper context properties
chrispader 161e66b
revert changes in AuthScreens
chrispader 18958b2
feat: add flag to ScreenWrapper
chrispader 2c420fa
Update AuthScreens.tsx
chrispader 4f8c568
fix: update comments of flags
chrispader 3789062
feat: implement `addBottomSafeAreaPaddingToContent`
chrispader fbb4e37
feat: extract contentContainerStyle logic with safe area padding and …
chrispader 4cf3a7c
fix: ESLint and TS errors
chrispader b4e830b
fix: contentContainerStyleProp might be nullish
chrispader 6f7657b
feat: implement `shouldCompensateForBottomSafeAreaPadding` flag to `K…
chrispader b584804
fix: always use edge-to-edge insets/paddings in `useContentContainerS…
chrispader 00f694f
feat: allow submit button to stick to the bottom of the screen in Forms
chrispader 9775d09
feat: add flag in `ScreenWrapper` to enable `KeyboardAvoidingView` bo…
chrispader b617141
fix: add FormElement styles to sticky submit button
chrispader 355eec3
fix: rename prop
chrispader 39d5ea4
fix: rename SelectionListProps type
chrispader b05db03
feat: implement bottom safe area handling in `FixedFooter` component
chrispader 2fc3edf
refactor: rename prop
chrispader 37ca0ae
feat: add flag for bottom sticky footer content
chrispader a7a91d2
fix: component names and spacing in KeyboardAvoidingView
chrispader 1e9c8e1
feat: add `shouldBlendOpacity` prop to `Button`
chrispader e72b319
fix: eslint errors
chrispader a0433b3
fix: comment
chrispader 788c8a4
feat: implement native function to detect soft-keys vs. gesture-bar o…
chrispader 15e38aa
refactor: rename hook
chrispader 2e72d04
feat: add default padding bottom for confirm button in SelectionList
chrispader 5b53463
fix: blend button automatically, when default footer content is used
chrispader ffc43a7
feat: add bottom safe area handling to BlockingViews
chrispader a9ef544
feat: pass addBottomSafeAreaPadding prop in EmptyStateComponent
chrispader 7484d61
fix: improve logic in useBottomSafeSafeAreaPaddingStyle
chrispader 0a5573f
fix: don't apply bottom safe area twice in SelectionList
chrispader c9b15c8
Update useBottomSafeSafeAreaPaddingStyle.ts
chrispader ac4dca5
rename param
chrispader e9cc422
fix: automatically blend opacity of footerContent in SelectionList
chrispader 8ec614e
fix: missing type declaration
chrispader c1d0964
fix: only apply bottom safe area padding when footer content sticky o…
chrispader 415656e
fix: improve offline indicator bottom safe area handling
chrispader 22e490e
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader f60d658
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader 574e9bd
refactor: simplify bottom content style in ScreenWrapper
chrispader 82d954c
fix: offline indicator simplification
chrispader 8786045
fix: rename variable
chrispader 42d5b74
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader 0719cdb
fix: Button dependency array
chrispader 98ff98b
fix: make ScreenWrapper use KeyboardAvoidingView offset by default
chrispader f69747a
refactor: rename `useStyledSafeAreaInsets` hook and update comments/docs
chrispader 33423bd
fix: eslint errors
chrispader 4404a99
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader 044c1de
fix: more es lint errors
chrispader b73aa0b
fix: wrong hook import
chrispader 12531a9
fix: prettier
chrispader e796192
Update BlockingView.tsx
chrispader 3f38b59
Update FullPageNotFoundView.tsx
chrispader 27eccb1
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader 42cc412
fix: address issues regarding prop JSDoc and additional comments
chrispader 0a678d0
Update src/components/SectionList/BaseSectionList.tsx
chrispader de9fac7
Update src/components/KeyboardAvoidingView/BaseKeyboardAvoidingView/i…
chrispader 2b10eb5
fix: address more issues
chrispader 2c44891
fix: improve styling code in Button component
chrispader 714997c
fix: remove `shouldFooterContentStickToBottom ` prop from SelectionList
chrispader 14fbe29
chore(deps): update `react-native-keyboard-controller`
chrispader edd6745
Revert "chore(deps): update `react-native-keyboard-controller`"
chrispader a2fccdd
Merge branch 'main' into pr/57128
chrispader 7a9be91
fix: remove trailing semicolon in Kotlin file
chrispader 22fbc11
fix: extract style util function type and fix import
chrispader 51acd01
Merge branch 'main' into pr/57281
chrispader d23c715
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader 7d27607
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader f525144
fix: old artifact after merge
chrispader 61a15da
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader a804499
fix: prettier
chrispader ebff006
fix: more prettier
chrispader afe075a
Merge branch 'main' into @chrispader/add-flags-for-disabling-bottom-s…
chrispader a6644d7
fix: side pane imports
chrispader File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to make these changes to the Mobile-Expensify?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checking on it!