On 2019-11-11 17:48, Andrew Lutsenko wrote:

> Hi Seth, Jeff, 
> I see you have touched this code recently. I've noticed that tabs are handled 
> differently for bbox calculation resulting in slightly oversized bbox. 
> 
> When rendering tabs we count them as whatever width is needed to align to 
> multiple of 4 and  +1 additional space but when calculating bbox it is align 
> to 4 spaces + '?' width, which is a bit wider. 
> 
> I attached a patch that fixes this particular issue to count tab as align to 
> 4 spaces + 1 space. 
> But I think this is confusing in general, why do we still render another 
> space after aligning width to multiple of 4? 
> Here is a picture of left-justified text in pcbnew. First line is "a<tab>b", 
> second "<4 spaces>b", third "<5 spaces>b". 
> 
> Regards, 
> Andrew

Good catch.  I hadn't noticed that.  The intention (in the comments) was
to align to the 4th column.  I pushed a fix as well as yours to the
code.  Thank you for your contribution to KiCad! 

Best- 

Seth 

KiCad Services Corporation 

                Seth Hillbrand

                LEAD DEVELOPER

                +1-530-302-5483‬ [1]

                Davis, CA

                www.kipro-pcb.com [2]    i...@kipro-pcb.com

                https://twitter.com/KiProEDA [3]
                https://www.linkedin.com/company/kicad [4]

 

Links:
------
[1] tel:+12126039372
[2] http://www.kipro-pcb.com/
[3] https://twitter.com/KiProEDA
[4] https://www.linkedin.com/company/kicad
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to