Re: [PATCH/RFC (take 2)] gitweb: New improved patchset view

--- Jakub Narebski <jnareb@gmail.com> wrote:

Replace "gitweb diff header" with its full sha1 of blobs and replace
it by "git diff" header and extended diff header. Change also somewhat
highlighting of diffs.

Changes:
* "gitweb diff header" which looked for example like below:
    file:_<sha1 before>_ -> file:_<sha1 after>_
  where 'file' is file type and '<sha1>' is full sha1 of blob is
  changed to
    diff --git _a/<file before>_ _b/<file after>_
  In both cases links are visible and use default link style. If file
  is added, a/<file> is not hyperlinked, if file is deleted, b/<file>
  is not hyperlinked.

"Everything clickable underlined" isn't the best way to represent things.
Anyway, my 2 cents is that I don't like the overly explicit underlineing.
I liked it the way it was in take 1.

* there is added "extended diff header", with <path> and <hash>
  hyperlinked (and <hash> shortened to 7 characters), and <mode>
  explained: '<mode>' is extended to '<mode> (<file type>)'.
* <file> hyperlinking should work also when <file> is originally
  quoted. For now we present filename quoted. This needed changes to
  parse_difftree_raw_line subroutine. And doesn't work: perhaps
  unquote is broken.

In which case we shouldn't commit this.  IOW, let's commit things
which we _know_ to work.

Why not resubmit your original patch with the bugfixes as few comments
as mentioned?

* from-file/to-file two-line header lines have slightly darker color
  than removed/added lines.
* chunk header has now delicate line above for easier finding chunk
  boundary, and top margin of 1px.

Wouldn't this be confusing with the other fine lines?
I personally don't like this chunk separation.  Chunk separation
already exists as is and we view it all the time elsewhere.

If you'd like to separate chunks, why not darken the background
of the section of line the chunk header is printed at?  I.e.
anything between the @@ including the @@.

    Luben