Recent commit changes RFC #1318

smashwilson · 2018-02-22T20:02:59Z

A more formal proposal of the "recent commits" view displayed beneath the mini commit editor.

Previous discussion and design ideating are available in #554.

Acceptance criteria

We will merge this RFC and begin work when we accumulate ✅ reviews from @smashwilson, @kuychaco, and @simurai.

Rendered

Implementation phases

Convert GitTabController and GitTabView to React. Port GitTabView and GitTabController to React #1319
List read-only commit information. Add read only recent commits view #1322
Replace the amend checkbox with the "undo" control. [WIP] Recent commit interactions #1328
Context menu with actions. Undo last commit and amend context menu option #1364
Show which commits have been pushed.
Balloon with action buttons and additional information.

kuychaco · 2018-02-23T01:59:47Z

docs/rfcs/001-recent-commits.md

+
+If the active repository has no commits yet, display a short panel with a background message: "Make your first commit".
+
+Otherwise, display a **recent commits** section containing a sequence of horizontal bars for each of the top three commits reachable from the current `HEAD`, with the most recently created commit on top. The user can resize the recent commits section. As it is expanded or shrunk, the number of visible commits is changed responsively.


After creating new branch, only display commit that was branched off of. This may be technically challenging to implement (how do we know when to do this? last reflog entry is a checkout?).

Oh hey there's a StackOverflow question about this:

git log mybranch --not $(git for-each-ref --format='%(refname)' refs/heads/ | grep -v "refs/heads/mybranch")

kuychaco · 2018-02-23T02:00:30Z

docs/rfcs/001-recent-commits.md

+
+On the most recent commit, display an "undo" button. Clicking "undo" performs a `git reset` and re-populates the commit message editor with the existing message.
+
+If any of the recent commits have been pushed to a remote, display a divider after the most recently pushed commit that shows an octocat icon. On hover, show the name of the remote tracking branch.


Additionally add dividers for other refs (branches and tags)

kuychaco · 2018-02-23T02:01:42Z

docs/rfcs/001-recent-commits.md

+2. List read-only commit information.
+3. Replace the amend checkbox with the "undo" control.
+4. Context menu with actions.
+5. Tooltip with action buttons and additional information.


Show which commits have been pushed

Surface info about other refs

kuychaco · 2018-02-23T02:12:50Z

docs/rfcs/001-recent-commits.md

+
+If the active repository has no commits yet, display a short panel with a background message: "Make your first commit".
+
+Otherwise, display a **recent commits** section containing a sequence of horizontal bars for each of the top three commits reachable from the current `HEAD`, with the most recently created commit on top. The user can resize the recent commits section. As it is expanded or shrunk, the number of visible commits is changed responsively.


Just show last three commits? Or allow seeing more via scrolling or dragging to make view larger?

Thanks to @simurai magic ✨ scrolling is a go. If we can, I'd really like to make it resizable by dragging too. If it's too technically challenging we can drop it, just like the "only show commits reachable by the current HEAD" bit up above, but I want to leave it in the RFC as a behavior goal if that's okay. I think I just feel like it'd be too frustrating not to be able to do 😁

kuychaco · 2018-02-23T02:18:50Z

docs/rfcs/001-recent-commits.md

+
+If the active repository has no commits yet, display a short panel with a background message: "Make your first commit".
+
+Otherwise, display a **recent commits** section containing a sequence of horizontal bars for each of the top three commits reachable from the current `HEAD`, with the most recently created commit on top. The user can resize the recent commits section. As it is expanded or shrunk, the number of visible commits is changed responsively.


Also show commits that are on remote branch but not local (as a result of auto-fetch). Punt to show in full-featured log and just link to that? Accordion style (expand/collapse). @simurai make mock-up to feel it out

kuychaco · 2018-02-23T02:28:07Z

docs/rfcs/001-recent-commits.md

+
+On the most recent commit, display an "undo" button. Clicking "undo" performs a `git reset` and re-populates the commit message editor with the existing message.
+
+If any of the recent commits have been pushed to a remote, display a divider after the most recently pushed commit that shows an octocat icon. On hover, show the name of the remote tracking branch.


@simurai would you mind making some quick mockups showing the dividers for remote tracking branch, local branch, tag. Also show icon and ref name visible (ex ref names: origin/feature-branch, master, v1.0.0) 🙏

consider when there are multiple refs for a given commit

smashwilson · 2018-02-23T02:38:21Z

docs/rfcs/001-recent-commits.md

+* For the most recent commit only, an "Amend" option. "Amend" is enabled if changes have been staged or the commit message mini-editor contains text. Choosing this applies the staged changes and modified commit message to the most recent commit, in a direct analogue to using `git commit --amend` from the command line.
+* A "Revert" option. Choosing this performs a `git revert` on the chosen commit.
+* A "Hard reset" option. Choosing this performs a `git reset --hard` which moves `HEAD` and the working copy to the chosen commit. When chosen, display a modal explaining that this action will discard commits and unstaged working directory context. Extra security: If there are unstaged working directory contents, artificially perform a dangling commit, disabling GPG if configured, before enacting the reset. This will record the dangling commit in the reflog for `HEAD` but not the branch itself.
+* A "Soft reset" option. Choosing this performs a `git reset --soft` which moves `HEAD` to the chosen commit and populates the staged changes list with all of the cumulative changes from all commits between the chosen one and the previous `HEAD`.


Plus: "mixed reset," consistent with the tool-tip actions 🎶

kuychaco · 2018-02-23T02:46:02Z

docs/rfcs/001-recent-commits.md

+* A "Hard reset" option. Choosing this performs a `git reset --hard` which moves `HEAD` and the working copy to the chosen commit. When chosen, display a modal explaining that this action will discard commits and unstaged working directory context. Extra security: If there are unstaged working directory contents, artificially perform a dangling commit, disabling GPG if configured, before enacting the reset. This will record the dangling commit in the reflog for `HEAD` but not the branch itself.
+* A "Soft reset" option. Choosing this performs a `git reset --soft` which moves `HEAD` to the chosen commit and populates the staged changes list with all of the cumulative changes from all commits between the chosen one and the previous `HEAD`.
+
+On click, reveal a tool-tip containing:


In the future when we have a multi-file diff view, we can consider just opening a pane item with this information

kuychaco · 2018-02-23T02:50:08Z

docs/rfcs/001-recent-commits.md

+- Providing a bridge to navigate to an expanded log view that allows more flexible and powerful history exploration.
+- Show an info icon and provide introductory information when no commits exist yet.
+- Add a "view diff from this commit" option to the recent commit context menu.
+- Integration with and navigation to "git log" or "git show" pane items when they exist.


How do we surface that we've made a commit for you before doing a git reset --hard? So you never actually lose any unstaged changes

Right-click context menu on history? "Undo reset" until next commit is made.

smashwilson · 2018-02-23T03:04:30Z

docs/rfcs/001-recent-commits.md

+2. List read-only commit information.
+3. Replace the amend checkbox with the "undo" control.
+4. Context menu with actions.
+5. Tooltip with action buttons and additional information.


"Balloon" instead of "tooltip" 🎈

simurai · 2018-02-27T10:42:20Z

Here a few dividers:

With label

Icon only

Tag

Hmm.. not sure. They use quite some space for such a small area and it's not clear if they are referring to the commits above or below a divider. Might fit better for the more fully-featured history log panel.

Alternatively we could have up/down arrows on the commits, matching the ahead/behind in the status-bar:

And tags could also be attached to a commit:

smashwilson · 2018-02-28T17:00:38Z

Ohh, fancy ✨

Looking over the options, some thoughts:

I'd agree that the octocat divider and the dedicated tag line both occupy a little too much vertical real estate for the information they give.
The up and down arrows are interesting! I like that they preserve the context of which commits are only local and only remote even if when the remote tracking branch ref isn't visible, and the symmetry with the push and pull buttons.
I really like the bubble-ref in your last .gif - in fact, I think I prefer that to any of the other full-line dividers. We do need to be sure that it scales correctly when many refs refer to a single commit. Maybe after we reach some low ref threshold (two?) they get their own horizontal row above the message? Something like this:

smashwilson · 2018-02-28T17:02:40Z

I've updated the writeup based on the review comments. @kuychaco, @simurai, care to have another look and review when you get the chance? 🙇

ampinsk · 2018-02-28T22:12:10Z

Hey all! @donokuda asked me to take a look here because of the overlap in problem space with some things we're working on at GHD.

Taking a look at the options you have here, I really like the icons direction. I think it does the best job working within the small timeline here. But, I think there might be a clearer icon to use than the up/down arrow. To me at least, the up/down arrow is pretty strongly associated with the push/pull action, not necessarily with the commit itself in terms of "this commit is ahead of my branch"

It would be awesome to come up with a new icon that we can use (here and in GHD!) to signal "this commit is ahead/behind your branch" Maybe something like |→ and ←| ?

Hope this is helpful! Let me know if I can clarify anything :)

simurai · 2018-03-01T02:34:58Z

@smashwilson We do need to be sure that it scales correctly when many refs refer to a single commit.

Here a few refs below and aligned with the message:

If they would be above, it might look like they belong to the commit above. We could also add divider lines for each commit, but then it quickly looks more heavy.

Hmm... also I think having these refs on a 2nd line makes messages less scannable because you get interrupted with something different when scrolling. How important is it to see these refs in the recent commits? I can see two options:

Only show tags (in the same line after the message). Their length (no more than vX.X.X) is somewhat predictable.
Show other refs on a 2nd line, but then disable scrolling.

simurai · 2018-03-01T02:59:25Z

@ampinsk Maybe something like |→ and ←| ?

👍 Yeah, a hint of their meaning could come from how branches show the ahead/behind:

The up/down arrows would be more from a local/remote perspective. Like this cheesy way of thinking that the remote is somewhere up in the cloud. Kinda like downloading and uploading.

commit ⬇︎ (you're behind: pull this down)
---- remote / local -----
commit ⬆︎ (you're ahead: push this up)
commit ⬆︎ (you're ahead: push this up)
commit

Looking at the up/down arrows, people might try to click them to actually push/pull? 🤔 Here with clouds around the arrows:

simurai · 2018-03-01T12:58:55Z

docs/rfcs/001-recent-commits.md

+3. Replace the amend checkbox with the "undo" control.
+4. Context menu with actions.
+5. Balloon with action buttons and additional information.
+6. Show which commits have been pushed.


I think it should say ~~have been~~ "can be" pushed. And also show commits that can be pulled (after fetching).

And also move it up in the priority list, maybe as far as 3rd place?

simurai · 2018-03-02T08:40:17Z

Here more feedback from @ampinsk + @donokuda. ❤️

When reloading Atom, it shows the "Make your first commit" for 1-2 seconds before showing the recent commits.
An alternative to the ahead/behind icons -> Have greyed out/bold text, different background color, or some other kind of highlight
Controversial: Instead of showing avatars -> show commit icon, with a vertical line like .com. This allows to have different states or icons like ahead/behind.
Don't show refs! Takes up too much space makes it harder to scan the messages.

And here the 2nd suggestion visualized:

kuychaco · 2018-03-28T22:54:21Z

We've completed Phases 1 and 2 with Phase 3 almost complete. With that we've reached our near-term goals for this feature 🎉

The rest is icing and we'll hold off until future quarters in order to prioritize progress on other roadmap items. I'll go ahead and merge this now so that we can save all of the goodness to refer to when we pick this back up again.

Thanks everyone for your fabulous input!

smashwilson added 2 commits February 22, 2018 11:56

"Borrow" the RFC template from atom/teletype with minor changes

e9fd65d

Work so far

429ac59

smashwilson added the feature request Propose new features or design label Feb 22, 2018

smashwilson added 9 commits February 22, 2018 12:04

HI @kuychaco!!!

56bebda

HI AGAIN @kuychaco!!!!

8032a5c

Note behavior within the bottom dock

60b454b

Hover -> click

527e6e6

(shhhhhh)

809aa7f

Pictures! 📸

ba8dd34

Implementation phases.

fee9114

Add the React conversion

486ff30

Merge branch 'master' into rfc-recent-commits

518c589

kuychaco reviewed Feb 23, 2018

View reviewed changes

smashwilson commented Feb 23, 2018

View reviewed changes

kuychaco reviewed Feb 23, 2018

View reviewed changes

smashwilson commented Feb 23, 2018

View reviewed changes

smashwilson mentioned this pull request Feb 23, 2018

Port GitTabView and GitTabController to React #1319

Merged

kuychaco mentioned this pull request Feb 24, 2018

Add read only recent commits view #1322

Merged

9 tasks

Update to account for review comments

99971cc

simurai added 2 commits March 1, 2018 21:50

Add titles to Explanation

f696da2

Make Explanation titles stronger

8f83aa4

simurai reviewed Mar 1, 2018

View reviewed changes

kuychaco merged commit fea70d2 into master Mar 28, 2018

kuychaco deleted the rfc-recent-commits branch March 28, 2018 22:54

This was referenced Apr 23, 2018

Update recent commits RFC #1413

Merged

More recent commits RFC edits #1420

Merged


		If the active repository has no commits yet, display a short panel with a background message: "Make your first commit".

		Otherwise, display a recent commits section containing a sequence of horizontal bars for each of the top three commits reachable from the current `HEAD`, with the most recently created commit on top. The user can resize the recent commits section. As it is expanded or shrunk, the number of visible commits is changed responsively.


		On the most recent commit, display an "undo" button. Clicking "undo" performs a `git reset` and re-populates the commit message editor with the existing message.

		If any of the recent commits have been pushed to a remote, display a divider after the most recently pushed commit that shows an octocat icon. On hover, show the name of the remote tracking branch.

Recent commit changes RFC #1318

Recent commit changes RFC #1318

Uh oh!

Conversation

smashwilson commented Feb 22, 2018 • edited by simurai Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Acceptance criteria

Implementation phases

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kuychaco Feb 23, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kuychaco Feb 23, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

simurai commented Feb 27, 2018

With label

Icon only

Tag

Uh oh!

smashwilson commented Feb 28, 2018

Uh oh!

smashwilson commented Feb 28, 2018

Uh oh!

ampinsk commented Feb 28, 2018

Uh oh!

simurai commented Mar 1, 2018

Uh oh!

simurai commented Mar 1, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

simurai commented Mar 2, 2018

Uh oh!

kuychaco commented Mar 28, 2018

Uh oh!

Uh oh!

smashwilson commented Feb 22, 2018 •

edited by simurai

Loading

kuychaco Feb 23, 2018 •

edited

Loading

kuychaco Feb 23, 2018 •

edited

Loading

simurai commented Mar 1, 2018 •

edited

Loading