Improved Adaptive Navigation in RYScaffold #877
Conversation
Updated several pages and components to use the Adaptive Navigation Suite for a more responsive and consistent UI layout.
Modify RYScaffold to use NavigationSuiteScaffold only when navigationSuiteItems is not null, otherwise default to using Scaffold. This ensures appropriate usage of components based on the availability of navigationSuiteItems while maintaining UI consistency.
|
Do you have any screenshot or screen record for this change? |
|
|
This's true, but instead of just replacing the component, I'd prefer a redesigned layout for large screens. And your screenshot looks no better than a bottom navigation bar ux wise imo |
|
Yes I also think that it should not be merged like that, but I thought that it was already a good starting point |
|
I agree that we should probably make a better navigation for large screens. However, I think the bottom bar here doesn't apply to the guidelines because it's a filter bar, not a navigation bar. It changes the current state of the screen instead of navigating users to another screen |
Summary
This PR improves RYScaffold to better handle adaptive navigation
Conditional Bottom Bar: Uses NavigationSuiteScaffold only if navigationSuiteItems are present, avoiding empty bars on mobile.
Adaptive Integration: Updated FilterBar for adaptive behavior based on screen size.
Future Improvement
Consider repositioning the settings button on the home page.