From: "Michał Kiedrowicz" <michal.kiedrowicz@gmail.com>
To: git@vger.kernel.org
Cc: "Jakub Narebski" <jnareb@gmail.com>, "Jeff King" <peff@peff.net>,
"Michał Kiedrowicz" <michal.kiedrowicz@gmail.com>
Subject: [PATCH v2 0/8] gitweb: Highlight interesting parts of diff
Date: Fri, 23 Mar 2012 23:56:49 +0100 [thread overview]
Message-ID: <1332543417-19664-1-git-send-email-michal.kiedrowicz@gmail.com> (raw)
Reading diff output is sometimes very hard, even if it's colored, especially if
lines differ only in few characters. This is often true when a commit fixes a
typo or renames some variables or functions.
This patch series teaches gitweb to highlight fragments that are different
between old and new line. This should mimic the same feature in Trac or GitHub.
Changes since v1:
* Rebased on origin/master.
* Added two patches at the beginning of the series. First one is a small fix
and second one adds a feature to esc_html_hl_regions() that will be needed
by later patches.
* Squashed patch "gitweb: Format diff lines just before printing" and reworked
changes to format_diff_line(). Now it accepts both raw line that needs to be
html_esc()'ed and already escaped line.
* Squashed changes in CSS with previous patch as proposed by Jakub.
* Dropped esc_html_mark_change() and started using esc_html_hl_regions()
because it was merged to origin/master.
* Fixed highlightning for combined diffs with more than two parents.
* Fixed indentation (tabs -> spaces) in conditions.
* Ensured that patches with "\ No newline at end of file" are properly
displayed.
* Ensured that binary patches are not supported in HTML format, thus this
series canot break it :) (answering Jakub's questions)
* Reworded some commit messages and fixed typos.
* Renamed few variables after comments from Jakub.
* Added some comments to the code.
* Added Acked-by from Jakub for few patches.
Michał Kiedrowicz (8):
gitweb: esc_html_hl_regions(): Don't create empty <span> elements
gitweb: Pass esc_html_hl_regions() options to esc_html()
gitweb: Extract print_sidebyside_diff_lines()
gitweb: Use print_diff_chunk() for both side-by-side and inline diffs
gitweb: Move HTML-formatting diff line back to process_diff_line()
gitweb: Push formatting diff lines to print_diff_chunk()
gitweb: Highlight interesting parts of diff
gitweb: Refinement highlightning in combined diffs
gitweb/gitweb.perl | 299 +++++++++++++++++++++++++++++++++-------------
gitweb/static/gitweb.css | 8 ++
2 files changed, 225 insertions(+), 82 deletions(-)
--
1.7.8.4
next reply other threads:[~2012-03-23 22:57 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-23 22:56 Michał Kiedrowicz [this message]
2012-03-23 22:56 ` [PATCH v2 1/8] gitweb: esc_html_hl_regions(): Don't create empty <span> elements Michał Kiedrowicz
2012-03-24 18:58 ` Jakub Narebski
2012-03-24 23:38 ` Michał Kiedrowicz
2012-03-29 18:04 ` [PATCH] gitweb: Use descriptive names in esc_html_hl_regions() Michał Kiedrowicz
2012-03-23 22:56 ` [PATCH v2 2/8] gitweb: Pass esc_html_hl_regions() options to esc_html() Michał Kiedrowicz
2012-03-24 19:15 ` Jakub Narebski
2012-03-24 23:31 ` Michał Kiedrowicz
2012-03-23 22:56 ` [PATCH v2 3/8] gitweb: Extract print_sidebyside_diff_lines() Michał Kiedrowicz
2012-03-28 14:33 ` Jakub Narebski
2012-03-29 17:25 ` Michał Kiedrowicz
2012-03-30 13:37 ` Jakub Narebski
2012-03-23 22:56 ` [PATCH v2 4/8] gitweb: Use print_diff_chunk() for both side-by-side and inline diffs Michał Kiedrowicz
2012-03-28 15:56 ` Jakub Narebski
2012-03-29 17:31 ` Michał Kiedrowicz
2012-03-30 13:34 ` Jakub Narebski
2012-03-30 13:37 ` Michal Kiedrowicz
2012-03-23 22:56 ` [PATCH v2 5/8] gitweb: Move HTML-formatting diff line back to process_diff_line() Michał Kiedrowicz
2012-03-29 16:14 ` Jakub Narebski
2012-03-29 16:49 ` Jakub Narebski
2012-03-29 17:36 ` Michał Kiedrowicz
2012-03-23 22:56 ` [PATCH v2 6/8] gitweb: Push formatting diff lines to print_diff_chunk() Michał Kiedrowicz
2012-03-29 16:59 ` Jakub Narebski
2012-03-29 17:41 ` Michał Kiedrowicz
2012-03-23 22:56 ` [PATCH v2 7/8] gitweb: Highlight interesting parts of diff Michał Kiedrowicz
2012-03-29 19:42 ` Jakub Narebski
2012-03-29 19:59 ` Michał Kiedrowicz
2012-03-23 22:56 ` [PATCH v2 8/8] gitweb: Refinement highlightning in combined diffs Michał Kiedrowicz
2012-03-29 23:37 ` Jakub Narebski
2012-03-30 6:49 ` Michal Kiedrowicz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1332543417-19664-1-git-send-email-michal.kiedrowicz@gmail.com \
--to=michal.kiedrowicz@gmail.com \
--cc=git@vger.kernel.org \
--cc=jnareb@gmail.com \
--cc=peff@peff.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).