Re: [PATCH] gitweb.css: Courer fonts for commits and tree-diff

[Luben Tuikov <ltuikov@yahoo.com>]

--- Junio C Hamano <junkio@cox.net> wrote:
> Luben Tuikov <ltuikov@yahoo.com> writes:
> 
> > Courer fonts for the commit header, commit message,
> > and tree-diff.
> >
> > Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>
> 
> I really do not want to be in the position to judge a patch like
> this, whose evaluation is solely based on "prettiness" factor.
> 
> But if I really have to, I would say this makes things uglier
> and less readable.  Maybe asking for monospace is less yucky but
> naming Courier explicitly?

Sometimes people would put ASCII art in their commit messages,
things like a simple table, compiler output, log output, etc,
and I wanted to _preserve_ the intent of such output.  This is why
I changed it to Courier. (sorry for the misspelling of Courier)

> BTW what tool do you use to generate and send your patches?  I
> remember another patch from you recently did not apply and it
> turned out the problem was that the last hunk had line numbers
> wrong.  This patch has exactly the same problem and I am
> wondering why.

I use git.  Those patches are against a one-off branch, which I created
for the purpose of sending you the patches.

The original patches are against my "git-lt" branch, which has
the File::MMagic changes in it -- it is my working branch.
So I generated a patch off of "git-lt", applied it to "one-off-gitweb",
which is identical to the git "master" branch, and sent the patches.

      Luben


> 
> I count 10 lines of original and 15 lines of new material in
> this hunk but the hunk header claims to have 11 lines of
> original text.
> 
> ---
>  gitweb/gitweb.css |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
> diff --git a/gitweb/gitweb.css b/gitweb/gitweb.css
> index 98410f5..b51282b 100644
> --- a/gitweb/gitweb.css
> +++ b/gitweb/gitweb.css
>  ... 
> @@ -142,11 +144,15 @@ table {
>  	padding: 8px 4px;
>  }
>  
> -table.project_list, table.diff_tree {
> +table.project_list {
>  	border-spacing: 0;
>  }
>  
> +table.diff_tree {
> +	border-spacing: 0;
> +	font-family: courier;
> +}
> +
>  table.blame {
>  	border-collapse: collapse;
>  }
> -- 
> 1.4.1.g9ca3
> 
> 
> -
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>