[RFC] branchview-NG

Bill Barry after.fallout at gmail.com
Tue Jun 17 16:30:02 CDT 2008


sorry for the spam, should have thought about it for another 10 seconds 
before sending the last message...

Why not just make the left padding on the li elements the width of the 
canvas?

Matt Mackall wrote:
> On Tue, 2008-06-17 at 23:05 +0200, Dirkjan Ochtman wrote:
>   
>> Matt Mackall wrote:
>>     
>>> Great. There's one visual glitch left, which is that there's whitespace
>>> between the ruling lines and the canvas. Other than that, I'd say it's
>>> ready to merge.
>>>       
>> Fixed this.
>>
>>     
>>> Oh, one other little glitch.. the part of the description where the hash
>>> was isn't clickable.
>>>       
>> Ah... That's a bit of a problem. Before, I had the ul layed out on top 
>> of the canvas. To get the row coloring right, I changed it around so 
>> that the ul is on the bottom with the canvas on top (otherwise the 
>> background color obscures the canvas). Apparently, the canvas makes the 
>> links unclickable. This only affects the left part because the canvas 
>> isn't as wide as the ul. At this point, there are 2 solutions:
>>
>> - Draw rectangles on the canvas for each row. a little ugly, because 
>> it's bad for performance, and might be brittle because it has to match 
>> up very close with the height of the actual list items.
>>
>> - Use two lists: one that has the background coloring, below the canvas, 
>> and one that has the text, above the canvas.
>>     
>
> That's probably fine. Here's a third idea: now that we have the row
> thingies, we can line up the titles to the right of the canvas.
>
>   

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://selenic.com/pipermail/mercurial-devel/attachments/20080617/6d05cf43/attachment.htm 


More information about the Mercurial-devel mailing list