Flutter Drawer won't close on tap with Navigator.pop()

Question

I have been playing around with the drawer and the items inside change some buttons in the body of the scaffold.

What can I do to have this drawer close when the user clicks on an item?

I have used the Navigator.POP(context); however it will not close the drawer.

Here is my code:

class MyApp extends StatefulWidget {
  const MyApp({Key? key}) : super(key: key);


  @override
  State<StatefulWidget> createState() {
    return _MyAppState();
  }
}

class _MyAppState extends State<MyApp> {
  int _modIndex = 0;


  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        drawer: Drawer(
          child: ListView(
            children: [
              DrawerHeader(child: Text('Choose a mod'),),
              ListTile(
                title: const Text('Wheels'),
                onTap: () {
                  setState(() {
                    _modIndex = 0;
                  });
                  Navigator.pop(context);
                },
              ),
              ListTile(
                title: const Text('Suspension'),
                onTap: () {
                  setState(() {
                    _modIndex = 1;
                  });
                  Navigator.pop(context);
                },
              ),
              ListTile(
                title: const Text('Body Colour'),
                onTap: () {
                  setState(() {
                    _modIndex = 2;
                  });
                  Navigator.pop(context);
                },
              )
            ],
          ),
      ...

I'm not sure what is wrong.

Md. Yeasin Sheikh · Answer 1 · 2021-12-06T16:03:58.027

0

this error can still happens when you use a context that is a parent of MaterialApp/WidgetApp.
This happens because when you do Navigator.of(context), it will start from the widget associated to the context used. And then go upward in the widget tree until it either find a Navigator or there's no more widget. read more

Separate MaterialApp from Home widget, means create another widget to have context.


class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(home: HomeWidget());
  }
}


class HomeWidget extends StatefulWidget {
  HomeWidget({Key? key}) : super(key: key);

  @override
  _HomeWidgetState createState() => _HomeWidgetState();
}

class _HomeWidgetState extends State<HomeWidget> {
  int _modIndex = 0;
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: Text("title"),
        ),
        body: Center(
          child: Text("_modIndex $_modIndex "),
        ),
        drawer: Drawer(
          child: ListView(
            children: [
              DrawerHeader(
                child: Text('Choose a mod'),
              ),
              ListTile(
                title: const Text('Wheels'),
                onTap: () {
                  setState(() {
                    _modIndex = 0;
                  });
                  Navigator.pop(context);
                },
              ),
              ListTile(
                title: const Text('Suspension'),
                onTap: () {
                  setState(() {
                    _modIndex = 1;
                  });
                  Navigator.pop(context);
                },
              ),
              ListTile(
                title: const Text('Body Colour'),
                onTap: () {
                  setState(() {
                    _modIndex = 2;
                  });
                  Navigator.pop(context);
                },
              )
            ],
          ),
        ));
  }
}

edited Dec 06 '21 at 16:03

answered Dec 06 '21 at 15:37

Md. Yeasin Sheikh

54,221
7
29
56

Thank you for your reply, although I now get a black screen after an item in the drawer is tapped. – Corden Dec 06 '21 at 15:53
Commented 1st option , check update – Md. Yeasin Sheikh Dec 06 '21 at 15:56
I wrote out the code myself so I always had the `_modIndex` uncommented. I still had the back screen though. – Corden Dec 06 '21 at 16:00
Are you talking about the body? It is because there is no `body` or `appBar` on `homePahe`, include those on `Scaffold`. check update – Md. Yeasin Sheikh Dec 06 '21 at 16:00
Yeah, everything looks as it should, I have Scaffold set up with the app bar, etc. The problem is when I now click an item in the drawer, the screen goes black. – Corden Dec 06 '21 at 16:06
`screen goes black` means drawer close event? – Md. Yeasin Sheikh Dec 06 '21 at 16:07
Yes? After a hot reload I get a red screen error. "Failed assertion: line 5339 pos 12: '_history.isNotEmpty': is not true." – Corden Dec 06 '21 at 16:13
do a `flutter clean` and rebuild the app again, – Md. Yeasin Sheikh Dec 06 '21 at 16:14
How do I do perform a flutter clean? I'm so sorry I'm really new to this. – Corden Dec 06 '21 at 16:17
on terminal type `flutter clean` and run the app again, and can you include what are you trying to archive – Md. Yeasin Sheikh Dec 06 '21 at 19:10

score 0 · Answer 2 · answered Dec 06 '21 at 18:57

First create a ScaffoldState key

GlobalKey<ScaffoldState> _scaffoldKey = new GlobalKey<ScaffoldState>();

Scaffold(
  key: _scaffoldKey,)

Now you can close the drawer using the closeDrawer() method.

closeDrawer() async {
  if (_scaffoldKey.currentState.isDrawerOpen) {
    _scaffoldKey.currentState.openEndDrawer();
  }
}

Flutter Drawer won't close on tap with Navigator.pop()

2 Answers2