git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/3] gitweb: 'blame' view improvements

From
Jakub Narebski <jnareb@gmail.com>
Date
Jul 10, 2009, 21:54 UTC
Message-ID
<200907102354.43232.jnareb@gmail.com>
This patch series was inspired by the study before sending proposal
for git-blame improvements; I have noticed that "previous"/"parent"
blame header was already implemented, just not documented:
  Subject: git-blame: Documenting "previous" header
  Message-ID: <200907071423.40938.jnareb@gmail.com>
  http://thread.gmane.org/gmane.comp.version-control.git/122837
Therefore proposal got cut down to tree blame:
  Subject: [RFC] Tree blame (git blame <directory>)
  Message-ID: <200907071058.39390.jnareb@gmail.com>
  http://thread.gmane.org/gmane.comp.version-control.git/122830

Using "previous" header should improve gitweb performance and (as I have noticed during implementing it) allow to follow 'linenr' links correctly through rename in a blamed commit.

Marking "boundary" somewhat was required to make it possible to distinguish which 'linenr' links would lead to parent of a blame commit (to previous version of a file), and which would elad to blamed commit. Current styling is up to debate.

After implementing using "previous" header in 'blame' view (which would allow to do the same for proposed 'blame_incremental' view) I remembered that last version of implementing 'blame_incremental' added author initials a'la "git gui blame". So I did the same for non-incremental 'blame' view. Exact formatting is up to debate.

Table of contents:
==================
 [PATCH 1/3] gitweb: Mark boundary commits in 'blame' view
 [PATCH 2/3] gitweb: Use "previous" header of git-blame -p in 'blame' view
 [PATCH 3/3] gitweb: Add author initials in 'blame' view, a la "git gui blame"
Jakub Narebski (3):
  gitweb: Mark boundary commits in 'blame' view
  gitweb: Use "previous" header of git-blame -p in 'blame' view
  gitweb: Add author initials in 'blame' view, a la "git gui blame"
 gitweb/gitweb.css  |    4 ++++
 gitweb/gitweb.perl |   51 ++++++++++++++++++++++++++++++++++++---------------
 2 files changed, 40 insertions(+), 15 deletions(-)
-- 
Jakub Narebski
Poland
Next: Jakub Narebski
Message 1 of 11 in “gitweb: 'blame' view improvements”
  1. 0/3 gitweb: 'blame' view improvementsJakub Narebski, Jul 10, 2009
  2. 1/3 gitweb: Mark boundary commits in 'blame' viewJakub Narebski, Jul 10, 2009
  3. 2/3 gitweb: Use "previous" header of git-blame -p in 'blame' viewJakub Narebski, Jul 10, 2009
  4. Junio C HamanoJul 10, 2009
  5. Jakub NarebskiJul 11, 2009
  6. Luben TuikovJul 12, 2009
  7. Jakub NarebskiJul 14, 2009
  8. 3/3 gitweb: Add author initials in 'blame' view, a la "git gui blame"Jakub Narebski, Jul 10, 2009
  9. Jakub NarebskiJul 11, 2009
  10. 5/3 gitweb: Incremental blame (proof of concept)Jakub Narebski, Jul 13, 2009
  11. 4/3 gitweb: Use light/dark class also in 'blame' viewJakub Narebski, Jul 12, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.