Skip to content

Gallery, Pesto: back button does nothing after going home from drawer #5103

Closed

Description

@sethladd
  • Open gallery
  • Tap pesto
  • Tap to open drawer
  • Tap home
  • Tap recipe
  • Hit the back button
  • Hit the back button
  • Hit the back button

On that third back button, I'd expect to be taken to the Gallery home. Instead, nothing happens.

On the fourth tap on the back button, I am taken back to the homescreen.

Activity

  1. added
    customer: galleryRelating to flutter/gallery repository. Please transfer non-framework issues there.
    on Jul 27, 2016
  2. added this to the August milestone on Jul 27, 2016
  3. mpcomplete commented on Jul 29, 2016

    @mpcomplete
    Contributor

    The problem with this one (even after Jason's crash fix) is that when you tap Home from the open Drawer, the Drawer is never closed - just destroyed. This leaves the drawer-opening on the navigation stack, so you need to hit back an extra time to skip past it.

    Maybe the Drawer should close itself when disposed.

  4. jason-simmons commented on Jul 30, 2016

    @jason-simmons
    Member

    This is happening because the tap of "Home" is considered a navigation that pushes another instance of Pesto onto the history stack.

    When you hit the back button, you navigate from the second instance of Pesto back to the first. You can see the change if you scroll the recipe list in the two instances to different places. In your example, the two instances looked identical, so it seems like the back button did nothing.

    If you're already in Home, then should another tap of Home in the drawer do anything?

  5. Hixie commented on Jul 30, 2016

    @Hixie
    Contributor

    @HansMuller was looking at this today.

    On Fri, Jul 29, 2016, 5:10 PM Jason Simmons [email protected]
    wrote:

    This is happening because the tap of "Home" is considered a navigation
    that pushes another instance of Pesto onto the history stack.

    When you hit the back button, you navigate from the second instance of
    Pesto back to the first. You can see the change if you scroll the recipe
    list in the two instances to different places. In your example, the two
    instances looked identical, so it seems like the back button did nothing.

    If you're already in Home, then should another tap of Home in the drawer
    do anything?

    —
    You are receiving this because you are subscribed to this thread.
    Reply to this email directly, view it on GitHub
    #5103 (comment),
    or mute the thread
    https://github.com/notifications/unsubscribe-auth/AAhpHD7J1_jbQY0JWe2e__ljI_5kbQEgks5qapaMgaJpZM4JWpVZ
    .

  6. self-assigned this
    on Aug 1, 2016
  7. HansMuller commented on Aug 1, 2016

    @HansMuller
    Contributor

    I have most of a fix. Hope to finish today. I filed a separate issue: #5159

  8. added a commit that references this issue on May 14, 2018
  9. github-actions commented on Sep 6, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  10. locked as resolved and limited conversation to collaborators on Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

customer: galleryRelating to flutter/gallery repository. Please transfer non-framework issues there.from: reviewReported in the press

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions