{"thread":{"id":"4856","subject":"[PATCH] gitweb.css: Courer fonts for commits and tree-diff","startedAt":"2006-07-12T03:43:23Z","lastAt":"2006-07-13T07:57:39Z","messageCount":4,"participants":["Luben Tuikov","Junio C Hamano","Matthias Lederhofer"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"23682","messageId":"20060712034323.48414.qmail@web31806.mail.mud.yahoo.com","threadId":"4856","inReplyTo":null,"subject":"[PATCH] gitweb.css: Courer fonts for commits and tree-diff","fromName":"Luben Tuikov","fromEmail":"ltuikov@yahoo.com","sentAt":"2006-07-12T03:43:23Z","receivedAt":"2006-07-12T03:43:23Z","isPatch":true,"sender":{"key":"ltuikov@yahoo.com","avatar":null},"body":"Courer fonts for the commit header, commit message,\nand tree-diff.\n\nSigned-off-by: Luben Tuikov \n---\n gitweb/gitweb.css | 9 ++++++++-\n 1 files changed, 8 insertions(+), 1 deletions(-)\n\ndiff --git a/gitweb/gitweb.css b/gitweb/gitweb.css\nindex 98410f5..b51282b 100644\n--- a/gitweb/gitweb.css\n+++ b/gitweb/gitweb.css\n@@ -60,6 +60,7 @@ div.page_footer_text {\n \n div.page_body {\n \tpadding: 8px;\n+\tfont-family: courier;\n }\n \n div.title, a.title {\n@@ -79,6 +80,7 @@ div.title_text {\n \tpadding: 6px 0px;\n \tborder: solid #d9d8d1;\n \tborder-width: 0px 0px 1px;\n+\tfont-family: courier;\n }\n \n div.log_body {\n@@ -142,11 +144,15 @@ table {\n \tpadding: 8px 4px;\n }\n \n-table.project_list, table.diff_tree {\n+table.project_list {\n \tborder-spacing: 0;\n }\n \n+table.diff_tree {\n+\tborder-spacing: 0;\n+\tfont-family: courier;\n+}\n+\n table.blame {\n \tborder-collapse: collapse;\n }\n-- \n1.4.1.g9ca3\n\n"},{"id":"23691","messageId":"7vac7f1htw.fsf@assigned-by-dhcp.cox.net","threadId":"4856","inReplyTo":"20060712034323.48414.qmail@web31806.mail.mud.yahoo.com","subject":"Re: [PATCH] gitweb.css: Courer fonts for commits and tree-diff","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-07-12T06:02:35Z","receivedAt":"2006-07-12T06:02:35Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Luben Tuikov writes:\n\n> Courer fonts for the commit header, commit message,\n> and tree-diff.\n>\n> Signed-off-by: Luben Tuikov \n\nI really do not want to be in the position to judge a patch like\nthis, whose evaluation is solely based on \"prettiness\" factor.\n\nBut if I really have to, I would say this makes things uglier\nand less readable. Maybe asking for monospace is less yucky but\nnaming Courier explicitly?\n\nBTW what tool do you use to generate and send your patches? I\nremember another patch from you recently did not apply and it\nturned out the problem was that the last hunk had line numbers\nwrong. This patch has exactly the same problem and I am\nwondering why.\n\nI count 10 lines of original and 15 lines of new material in\nthis hunk but the hunk header claims to have 11 lines of\noriginal text.\n\n---\n gitweb/gitweb.css | 9 ++++++++-\n 1 files changed, 8 insertions(+), 1 deletions(-)\ndiff --git a/gitweb/gitweb.css b/gitweb/gitweb.css\nindex 98410f5..b51282b 100644\n--- a/gitweb/gitweb.css\n+++ b/gitweb/gitweb.css\n ... \n@@ -142,11 +144,15 @@ table {\n \tpadding: 8px 4px;\n }\n \n-table.project_list, table.diff_tree {\n+table.project_list {\n \tborder-spacing: 0;\n }\n \n+table.diff_tree {\n+\tborder-spacing: 0;\n+\tfont-family: courier;\n+}\n+\n table.blame {\n \tborder-collapse: collapse;\n }\n-- \n1.4.1.g9ca3\n"},{"id":"23719","messageId":"20060712174610.71046.qmail@web31810.mail.mud.yahoo.com","threadId":"4856","inReplyTo":"7vac7f1htw.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] gitweb.css: Courer fonts for commits and tree-diff","fromName":"Luben Tuikov","fromEmail":"ltuikov@yahoo.com","sentAt":"2006-07-12T17:46:10Z","receivedAt":"2006-07-12T17:46:10Z","isPatch":true,"sender":{"key":"ltuikov@yahoo.com","avatar":null},"body":"--- Junio C Hamano wrote:\n> Luben Tuikov writes:\n> \n> > Courer fonts for the commit header, commit message,\n> > and tree-diff.\n> >\n> > Signed-off-by: Luben Tuikov \n> \n> I really do not want to be in the position to judge a patch like\n> this, whose evaluation is solely based on \"prettiness\" factor.\n> \n> But if I really have to, I would say this makes things uglier\n> and less readable. Maybe asking for monospace is less yucky but\n> naming Courier explicitly?\n\nSometimes people would put ASCII art in their commit messages,\nthings like a simple table, compiler output, log output, etc,\nand I wanted to _preserve_ the intent of such output. This is why\nI changed it to Courier. (sorry for the misspelling of Courier)\n\n> BTW what tool do you use to generate and send your patches? I\n> remember another patch from you recently did not apply and it\n> turned out the problem was that the last hunk had line numbers\n> wrong. This patch has exactly the same problem and I am\n> wondering why.\n\nI use git. Those patches are against a one-off branch, which I created\nfor the purpose of sending you the patches.\n\nThe original patches are against my \"git-lt\" branch, which has\nthe File::MMagic changes in it -- it is my working branch.\nSo I generated a patch off of \"git-lt\", applied it to \"one-off-gitweb\",\nwhich is identical to the git \"master\" branch, and sent the patches.\n\n Luben\n\n\n> \n> I count 10 lines of original and 15 lines of new material in\n> this hunk but the hunk header claims to have 11 lines of\n> original text.\n> \n> ---\n> gitweb/gitweb.css | 9 ++++++++-\n> 1 files changed, 8 insertions(+), 1 deletions(-)\n> diff --git a/gitweb/gitweb.css b/gitweb/gitweb.css\n> index 98410f5..b51282b 100644\n> --- a/gitweb/gitweb.css\n> +++ b/gitweb/gitweb.css\n> ... \n> @@ -142,11 +144,15 @@ table {\n> \tpadding: 8px 4px;\n> }\n> \n> -table.project_list, table.diff_tree {\n> +table.project_list {\n> \tborder-spacing: 0;\n> }\n> \n> +table.diff_tree {\n> +\tborder-spacing: 0;\n> +\tfont-family: courier;\n> +}\n> +\n> table.blame {\n> \tborder-collapse: collapse;\n> }\n> -- \n> 1.4.1.g9ca3\n> \n> \n> -\n> To unsubscribe from this list: send the line \"unsubscribe git\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at http://vger.kernel.org/majordomo-info.html\n> \n"},{"id":"23749","messageId":"E1G0w4p-0007c9-O3@moooo.ath.cx","threadId":"4856","inReplyTo":"20060712174610.71046.qmail@web31810.mail.mud.yahoo.com","subject":"Re: [PATCH] gitweb.css: Courer fonts for commits and tree-diff","fromName":"Matthias Lederhofer","fromEmail":"matled@gmx.net","sentAt":"2006-07-13T07:57:39Z","receivedAt":"2006-07-13T07:57:39Z","isPatch":true,"sender":{"key":"matled@gmx.net","avatar":null},"body":"Luben Tuikov wrote:\n> --- Junio C Hamano wrote:\n> > But if I really have to, I would say this makes things uglier\n> > and less readable. Maybe asking for monospace is less yucky but\n> > naming Courier explicitly?\n> \n> Sometimes people would put ASCII art in their commit messages,\n> things like a simple table, compiler output, log output, etc,\n> and I wanted to _preserve_ the intent of such output. This is why\n> I changed it to Courier. (sorry for the misspelling of Courier)\nmonospace should do this too, see[1]. I think it should be added as\nfallback[2] at least but I see no reason not to use font-family:\nmonospace.\n\n[1] http://www.w3.org/TR/REC-CSS2/fonts.html#monospace-def\n[2] http://www.w3.org/TR/REC-CSS2/fonts.html#generic-font-families\n"}]}