{"thread":{"id":"5868","subject":"Re: [RFC] gitweb wishlist and TODO list","startedAt":"2006-10-10T01:47:43Z","lastAt":"2006-10-10T08:54:54Z","messageCount":2,"participants":["Jakub Narebski","Luben Tuikov"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"28652","messageId":"20061010014743.80094.qmail@web31801.mail.mud.yahoo.com","threadId":"5868","inReplyTo":"egdge3$t12$1@sea.gmane.org","subject":"Re: [RFC] gitweb wishlist and TODO list","fromName":"Luben Tuikov","fromEmail":"ltuikov@yahoo.com","sentAt":"2006-10-10T01:47:43Z","receivedAt":"2006-10-10T01:47:43Z","isPatch":false,"sender":{"key":"ltuikov@yahoo.com","avatar":null},"body":"--- Jakub Narebski <jnareb@gmail.com> wrote:\n>  * Improve blame view, making use of --porcelain option to git-blame (for\n>    later). Perhaps change blame view from table based one to div based one.\n\n>    Use different colors for different commits (graph coloring problem).\n\nOh, no please no.\n\nWhy do you think I left the color list as a list?  I did try to use\nmore colors when I wrote it, and it was ugly as h3ll and very distracting\nwhen doing real work.  So I ended up with the two color (shades) we have\nnow and this is what I submitted.\n\nAlso, any kind of \"graph coloring problem\" would make blame slow.\n\nIn any way, if you/someone does implement this \"coloring\" can you please\nmake it an option, because I'll never turn it on.  Thanks!\n\n     Luben\n"},{"id":"28503","messageId":"200610101054.54948.jnareb@gmail.com","threadId":"5868","inReplyTo":"20061010014743.80094.qmail@web31801.mail.mud.yahoo.com","subject":"Re: [RFC] gitweb wishlist and TODO list","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2006-10-10T08:54:54Z","receivedAt":"2006-10-10T08:54:54Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Luben Tuikov wrote:\n> --- Jakub Narebski <jnareb@gmail.com> wrote:\n> >  * Improve blame view, making use of --porcelain option to git-blame (for\n> >    later). Perhaps change blame view from table based one to div based one.\n> \n> >    Use different colors for different commits (graph coloring problem).\n> \n> Oh, no please no.\n> \n> Why do you think I left the color list as a list?  I did try to use\n> more colors when I wrote it, and it was ugly as h3ll and very distracting\n> when doing real work.  So I ended up with the two color (shades) we have\n> now and this is what I submitted.\n> \n> Also, any kind of \"graph coloring problem\" would make blame slow.\n\nOne of ideas (without having some nice _mathematical_ solution, i.e. not\nhaving for to try and check different coloring but calculating coloring, \nof a blame graph coloring problem[*1*]) was to use few colors, 3, 6, 8\nbased on some hash of sha1 of commit (for example first character) plus\nalternating \"darkness\" of those colors to ensure that neighbours would\nhave different colors. Another was to use first 6 characters of sha1 as\na color, then flatten the color to suitable for background (perhaps also\nwith some kind of ensuring that neigbour blames would have different color). \n\nJunio idea of basing color/brighness (of some part of blame output at\nleast) on the _age_ of region (perhaps using two altrenating _colors_)\nhas also it's merit.\n\nNevertheless, such change would be preceded by an RFC, and discussion.\n\n> In any way, if you/someone does implement this \"coloring\" can you please\n> make it an option, because I'll never turn it on.  Thanks!\n\nNot a problem to make blame coloring a feature.\n\n\nFootnotes:\n[*1*] Blame graph coloring problem: 1) regions blamed on the same commit\nshould have the same color 2) neighbour blame regions should have different\ncolors.\n-- \nJakub Narebski\nPoland\n"}]}