[PATCH RFC 0/2] Patch page style changes

Nate Case ncase at xes-inc.com
Fri Nov 7 10:49:34 EST 2008


These patches make some changes to the patch display pages.  We've
been running with these changes on our in-house patchwork server
for a while without any problems.  Peter Tyser, who did most of the
work for this, sent these to Jeremy before in an earlier form.  This
version is rebased on top of the latest master branch.

Changes:

  (1) Make diff display style match gitweb
      Rationale: We think it looks nicer :)

  (2) Re-orders patch page to show the patch before the comments
      Rationale: I think this is more logical than the current format.
                 Currently, patchwork shows you the comments on the
                 patch and then the patch itself at the bottom.

                 The patch is the most important part of the page, and
                 should be shown first.  The current format reminds
                 me of top-posting.

  (3) Add a 'comments' column to the patch list indicating the
      number of replies.
      Rationale: It's sometimes useful to be able to identify patches
                 that haven't had any feedback yet.

I don't know if these changes will ever be accepted, but I thought
I'd post them so others on the list have the opportunity to use them
or provide comments.

--
Nate Case <ncase at xes-inc.com>



More information about the Patchwork mailing list