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

implement adaptive navigation, target sdk 35 and edge-to-edge, fill nav suite item when selected, and don't set transitions to null when trying to override + dependency updates and add/update .idea xmls #57

Conversation

soupslurpr
Copy link
Member

@soupslurpr soupslurpr commented Oct 27, 2024

Adaptive navigation + edge-to-edge support

Adaptive navigation allows for a better experience for various window sizes. The animation is also changed appropriately to a vertical sliding animation instead of a horizontal one. The AndroidX library androidx.compose.material3:material3-adaptive-navigation-suite is used to implement adaptive navigation.

Previously, edge-to-edge wouldn't have even been visible because a bottom navigation bar was always used despite the window size.

Pixel 9 Pro Fold (open)

Before

After

When closed, it automatically switches to a bar instead

Filled icons for selected nav suite items

The selected nav suite item will now switch to a filled icon variant when available. The filled icon is only available for the Community and Donate icons.

Before

After

@soupslurpr soupslurpr marked this pull request as draft October 27, 2024 06:33
@soupslurpr soupslurpr changed the title implement adaptive navigation, target sdk 35, fill nav suite item when selected, and don't set transitions to null when trying to override + dependency updates and add/update .idea xmls implement adaptive navigation, target sdk 35 and edge-to-edge, fill nav suite item when selected, and don't set transitions to null when trying to override + dependency updates and add/update .idea xmls Oct 27, 2024
@soupslurpr soupslurpr force-pushed the adaptive-navigation-+-dependency-updates-+-let-default-transitions-be-overriden branch from 1b44911 to 81e0c75 Compare October 27, 2024 08:44
@soupslurpr soupslurpr marked this pull request as ready for review October 27, 2024 08:54
@soupslurpr soupslurpr force-pushed the adaptive-navigation-+-dependency-updates-+-let-default-transitions-be-overriden branch from 81e0c75 to ef5f688 Compare October 27, 2024 09:12
@thestinger thestinger closed this Oct 27, 2024
@thestinger
Copy link
Member

Merged with verification metadata regenerated.

@soupslurpr soupslurpr deleted the adaptive-navigation-+-dependency-updates-+-let-default-transitions-be-overriden branch October 27, 2024 21:53
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