seveas/more_ui_tweaks #49

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
1 participant
@seveas
Contributor

seveas commented Aug 1, 2012

These are some small UI improvements based on feedback in #p5p after announcing http://perl5.git.perl.org:3000
(Take 2, now with 100% less leaks)

b021f85 Make the diff-tree snippet on the commit page useful
ad3ce84 Less confusing commit display
cff1a85 Commit message display tweaks

@seveas

This comment has been minimized.

Show comment Hide comment
@seveas

seveas Aug 1, 2012

Contributor

Hmm, that list of commits makes no sense -- the branch is based on 4ddefca, as in "upstream/master" in my clone

[edit: ah, you rebased your repo. Did the same now and pushed a fixed branch]

Contributor

seveas commented Aug 1, 2012

Hmm, that list of commits makes no sense -- the branch is based on 4ddefca, as in "upstream/master" in my clone

[edit: ah, you rebased your repo. Did the same now and pushed a fixed branch]

seveas added some commits Jul 30, 2012

Commit message display tweaks
- In shortlog/longlog/fancy diff short message now is a link to the commit
- Fancy diff now shows the entire message
Less confusing commit display
Commits with a message of more than 1 line looked like

 ID  Author timestamp  Message line 1  Author info

     Commit timestamp  Message line 2  Committer info
                       Message line 3
                       Message line 4
                       Message line 5

This is rather confusing. I've moved the message to the left and
combined author/committer info and timestamp so it now looks like

 ID  Message line 1  Author info

     Message line 2  Committer info
     Message line 3
     Message line 4
     Message line 5

Which is less confusing, at least to me.
Make the diff-tree snippet on the commit page useful
- Change the wording slightly
- Make the links link to the fancey diff page, instead of merely broken
  anchors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment