• Forbo@lemmy.ml
    link
    fedilink
    arrow-up
    17
    arrow-down
    1
    ·
    8 hours ago

    Problem is, there’s an entire generation of users that have gotten super used to “discard changes” as a means of signalling “on second thought, don’t do anything”.

    • AeonFelis@lemmy.world
      link
      fedilink
      English
      arrow-up
      1
      ·
      4 hours ago

      From a certain point of view - isn’t this exactly what happened here?

      I often go into a Git worktree of one of my projects and mess around a bit to try something out. If I find it’s not working, I tell git to discard the changes with git checkout . and git clean -df. What I’m saying is exactly “on second thought, don’t do anything" - while what happens in practice is that Git restores all files to their HEAD status and removes all the new files that are not already in HEAD.

      Of course, the difference is that I already have all the work I want to keep under source control, so these changes I’ve discarded really were that - just changes. He, on the other hand, “was just playing with the source control option” - so these “changes” he was discarding really were all his work. But Git did not know that.

    • ulterno@programming.dev
      link
      fedilink
      English
      arrow-up
      4
      ·
      7 hours ago

      That’s definitely how it is seen.
      If I were to see “Discard Changes” anywhere in a dialogue, I would assume it will discard whatever changes I made in that dialogue. In this case, probably some source control related changes. If it were to say “Warning: This will Discard ALL changes!!!”, I might do a double take, but had I never used git CLI before, I would still assume that at most it would discard “ALL” changes made in the current session.

      For me personally, I would consider it more useful for it to say:

      This action will delete the following files:
      - followed
      - by
      - a
      - list
      - of
      - files
      - that
      - would
      - be
      - deleted
      Continue?
      

      Which neither has to look like a warning, acting like you might be doing something you don’t want to and also is much more useful for someone like me who wants to double check what exactly I am deleting.


      Also, I have used git CLI before and apart from being able to see blame in the editor itself and maybe a better representation of tree, I don’t feel the need to use any git GUI tool. Even when I tried, I realised it was slower and more finicky to use. So, it would stand to reason that it should be targetted towards people who don’t use CLI (and might have never used git CLI).