Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Can't undo after discarding all hunks in a patch #606

Open

Description

@nathansobo

After discarding all hunks in a patch, the patch view is closed and there's no way to undo the discard. Perhaps the undo history for the staged file list should include a discard operation from the patch view if that discard operation removes the last hunk and causes it to close.

Versions

GitHub Package Version (git --git-dir ~/.atom/packages/github/.git rev-parse head):
cdf3b27

Activity

  1. kuychaco commented on Mar 13, 2017

    @kuychaco
    Contributor

    Yeah thought about this and I'm thinking that a decent UX could be to keep the diff view open in an empty state and display a button in the middle of the pane that takes the user to the staged diff view. How does that sound?

    Chatted with @simurai a bit about this before I believe. What're your 💭 s @simurai?

  2. simurai commented on Mar 14, 2017

    @simurai
    Contributor

    @kuychaco keep the diff view open in an empty state and display a button in the middle of the pane that takes the user to the staged diff view.

    👍

    "keep the diff view open" is also nice if you wanna do other stuff in the Git panel after discarding. Currently if you discard all hunks, the pane closes and your focus gets move back to the editor. But you might are not done yet and want to commit or select another file.

  3. nathansobo commented on Mar 14, 2017

    @nathansobo
    ContributorAuthor

    👍 for keeping it open.

  4. added this to the Public Release milestone on Mar 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions