Simon Wiles

Even better would be if this was handled by the stemmer, such that, e.g., `McDonald's` would actually be indexed as `McDonald` (and word positions would work properly).

There are multiple PRs open that address this issue (#72, #80, #92...).

> Would it be possible as a quick win to add a button for toggling all? Just an idea, not worth blocking this over. Yeah, I like the idea of...

The `numberOfSelectedPatchLines` variable was only used to reset the edit state when it evaluated to zero, so I dropped it and replace the check with a test against `this.patchLineList().filter(Boolean).length` (see...

### Important: When investigating this, I noticed a bug in the way that ungit operates in the present release version (`1.5.11`). See below for a quick demonstration: ...

Okay, this last commit addresses the bug in production, I think (although it's really a workaround rather than a proper fix).

It can be done like this: http://stackoverflow.com/a/19700341/1668622 (this is also represents a workaround which can be used in the context of ungit).

Revisiting this by chance, I notice that as of git 2.13 this looks much more approachable: https://github.com/git/git/blob/44c3f09fa501ccd83e3362080a99ad4b46cda127/Documentation/git-stash.txt#L65-L69

I've actually written a simple ~15 line change which allows clicking (or tapping) any part of the line to toggle the checkbox. I've was gonna tidy it up and offer...

Yeah, that's a common part of my workflow too, so I know that still works. I've just noticed I also have a local change that prevents the selection of an...