responsive_scaffold icon indicating copy to clipboard operation
responsive_scaffold copied to clipboard

[FeatureRequest] update to support NavigationRail

Open daadu opened this issue 5 years ago • 4 comments

Flutter 1.17 comes with NavigationRail component which is part of material design guideline. It suggest adapting BottomNavigationBar [mobile] -> NavigationRail [tablet] -> NavigationDrawer [desktop]. Check image below:

ezgif com-video-to-gif

daadu avatar May 16 '20 19:05 daadu

This package is my replacement: https://pub.dev/packages/navigation_rail

rodydavis avatar May 26 '20 20:05 rodydavis

I think this should be adapted with this package it self following material design guidelines. If there is a BottomBar defined then It should be replaces with NavigationRail in larger screen (check animation).

daadu avatar May 27 '20 04:05 daadu

It is my package I created too, but I agree. I can have this adapt it as well. This package was made before the rail was added to the spec. Responsive scaffold is a drop in scaffold replacement whereas navigation rail is a new layout widget similar I in feature set.

rodydavis avatar May 27 '20 04:05 rodydavis

Hi @rodydavis I need this functionality and am starting work on it. How would you like this feature to be implemented? Just wrapping the responsive_scaffold in a NavRail does not seem to be the way to go. How should the tablet view be handled (Currently the AppBar/titlebar continues over the nav rail when in tablet mode, see the picture). Should the NavigationRail be integrated into the master view? Should we just drop the appbar of the master and detail and show the unified one?. Also The Master view supports usage of Slivers so how should they be handled? Screenshot_20210520_113116

I'm deprecating my own NavRail and if I already do the work I'd like to contribute it back to the project :)

Leptopoda avatar May 20 '21 09:05 Leptopoda