← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix overbar drawing for non-italic text

 

+1 for the patch. The change seems sensible just by looking at the raw
diff and the presented images justify correctness.

Cheers,
Orson

On 06/23/2018 05:32 PM, Andrzej Wolski wrote:
> Wayne,
> 
> this is after patch:
> https://imgur.com/a/54ESCMD
> 
> Andrzej
> 
> W dniu 2018-06-23 o 16:54, Wayne Stambaugh pisze:
>> Andrzej,
>>
>> Do you have an image of the same characters after you patch?  The other
>> question I have is what happens when consecutive characters have and
>> overbar?  AFAIK, the overbar should be continuous (no breaks between
>> characters).  Did you test this as well.  If this needs fixed, I would
>> like to get it merged for rc3.  Anyone else have any thoughts on this.
>>
>> Cheers,
>>
>> Wayne
>>
>> On 06/22/2018 10:35 AM, Andrzej Wolski wrote:
>>> No, the patch didn't go the wrong way.
>>> If you try to shorten the right side of the bar, for some characters it
>>> will look really bad, and in case of "I" it will turn into dot.
>>> This is before patch:
>>> https://imgur.com/a/TjibR0Q
>>>
>>> Andrzej
>>>
>>>
>>> W dniu 2018-06-22 o 15:40, Kevin Cozens pisze:
>>>> On 2018-06-22 07:01 AM, Andrzej Wolski wrote:
>>>>> This is an illustration of what this patch do:
>>>>> https://imgur.com/a/GxIeLkA
>>>> Hm... did the patch go the wrong way with the change to the position
>>>> and length of the bar over non-italicized text? Before the bar started
>>>> at the left side of the letter and extends past the right side of the
>>>> letter. After it starts before the letter and extends past the right
>>>> side. Contrast that to italicized text where the bar starts and ends
>>>> based on the left-most and right-most part of the drawn letter.
>>>>
>>>> I think a patch should shorten the right-hand extent of the bar over
>>>> non-italicized text so it is only over the letter.
>>>>
>>>
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help   : https://help.launchpad.net/ListHelp
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp


Follow ups

References