kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #36301
Re: [PATCH] Fix overbar drawing for non-italic text
-
To:
<kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Suminski <maciej.suminski@xxxxxxx>
-
Date:
Sat, 23 Jun 2018 20:43:42 +0200
-
Authentication-results:
spf=pass (sender IP is 188.184.36.48) smtp.mailfrom=cern.ch; lists.launchpad.net; dkim=none (message not signed) header.d=none;lists.launchpad.net; dmarc=bestguesspass action=none header.from=cern.ch;
-
In-reply-to:
<3a3170e7-caaa-f969-b745-2d45eb4bfad4@gmail.com>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0
+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